💬 maflcko commented on pull request "fuzz: avoid underflow in `coins_view` target":
(https://github.com/bitcoin/bitcoin/pull/29164#issuecomment-1873918402)
```
fuzz: coins.cpp:338: void CCoinsViewCache::SanityCheck() const: Assertion `recomputed_usage == cachedCoinsUsage' failed.

💬 ismaelsadeeq commented on pull request "test: doc: follow-up #28368":
(https://github.com/bitcoin/bitcoin/pull/29013#discussion_r1439371178)
Thanks fixed now

💬 darosior commented on pull request "fuzz: avoid underflow in `coins_view` target":
(https://github.com/bitcoin/bitcoin/pull/29164#issuecomment-1873927193)
My bad, i ran the new `coins_db` introduced in #28216 for a while but not this one.. Will debug to find the other source of inconsistency.

💬 fanquake commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#issuecomment-1873927915)
> The macOS CI fails (as expected). Moving to draft for now, until someone clarifies the macOS clang situation. (I don't use macOS)

Xcode 14.3.1 (currently used in the CI) is based on ~ LLVM 14. It's failing because removing the fuzz workaround requires LLVM/Clang 15 or later. The GH image ships with newer Xcodes, https://github.com/actions/runner-images/blob/main/images/macos/macos-13-Readme.md#xcode, so I assume we could switch to using Xcode 15.x + in the CI (based on LLVM 16.x).

The mu
...

💬 fanquake commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#issuecomment-1873930799)
The libbitcoinkernel `clang-15 libc++` job is also failing, so I guess this isn't actually fixed with clang-15, regardless of macOS?

💬 maflcko commented on pull request "fuzz: avoid underflow in `coins_view` target":
(https://github.com/bitcoin/bitcoin/pull/29164#issuecomment-1873934953)
> The crash can be reproduced with the following seed:

On what commit? For me it seems to pass on current master.

💬 darosior commented on pull request "fuzz: avoid underflow in `coins_view` target":
(https://github.com/bitcoin/bitcoin/pull/29164#issuecomment-1873935640)
You need to add the sanity check to trigger it.

💬 hebasto commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#discussion_r1439389706)
```suggestion
sudo xcode-select --switch "/Applications/Xcode15.1.app"
```

💬 fanquake commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#discussion_r1439390908)
If any version is going to be picked, shouldn't it be the one we use for releases?

💬 hebasto commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#discussion_r1439391581)
Or `sudo xcode-select -switch /Applications/Xcode_15.1.app`. It works in https://github.com/hebasto/bitcoin/actions/runs/7385516810/job/20090426555

💬 fanquake commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#discussion_r1439391984)
> Run sudo xcode-select --switch "/Applications/Xcode15.1.app"
xcode-select: error: invalid developer directory '/Applications/Xcode15.1.app'

💬 darosior commented on pull request "fuzz: avoid underflow in `coins_view` target":
(https://github.com/bitcoin/bitcoin/pull/29164#issuecomment-1873966765)
Sigh... I'm tired of this target just doing random things. Turns out you can't perform the sanity checks because it's using a dummy backend where `Flush()` won't actually erase the cache. I'll just use a hack to avoid the crash in #28216.

✅ darosior closed a pull request: "fuzz: avoid underflow in `coins_view` target"
(https://github.com/bitcoin/bitcoin/pull/29164)

💬 darosior commented on pull request "fuzz: a new target for the coins database":
(https://github.com/bitcoin/bitcoin/pull/28216#issuecomment-1873968224)
Removed the use of a proper fix to simply abuse `Flush()` when the exception is hit to avoid the crash. See the first commit.

💬 hebasto commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#discussion_r1439419407)
> If any version is going to be picked, shouldn't it be the one we use for releases?

Xcode 15.0.1 (15A507) and 15.1 (15C65) versions are available now, and they are subjects for frequent updates.

💬 fanquake commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#discussion_r1439422328)
> Xcode 15.0.1 (15A507) and 15.1 (15C65) versions are available now,

My suggestion was to pick `15.0`, which we use for release builds, so that CI better matches our release process, rather than `15.1`, as you suggested. Otherwise code could compile in the CI, but not in Guix.

💬 maflcko commented on pull request "build: Bump clang minimum supported version to 15":
(https://github.com/bitcoin/bitcoin/pull/29165#discussion_r1439433156)
Looks like the macOS CI now fails for unrelated reasons. If someone created a separate pull request to bump xcode in the macOS CI, that'd be great.

💬 brunoerg commented on pull request "fuzz: a new target for the coins database":
(https://github.com/bitcoin/bitcoin/pull/28216#issuecomment-1874011124)
Concept ACK

💬 maflcko commented on pull request "miniscript: make operator""_mst consteval":
(https://github.com/bitcoin/bitcoin/pull/28657#issuecomment-1874012428)
Confirmed that this cuts a third of the runtime off of the fuzz input from https://github.com/bitcoin/bitcoin/pull/28832#issuecomment-1827538761

💬 fanquake commented on pull request "miniscript: make operator""_mst consteval":
(https://github.com/bitcoin/bitcoin/pull/28657#issuecomment-1874014418)
Maybe we should also retry the straight constexpr -> consteval change, without having to do all the refactoring for MSVC? Microsoft might have fixed their compiler.