MSVC 17.12.0 internal compiler error

hebasto commented at 10:47 am on November 17, 2024: member

The latest version of MSVC causes an internal compiler error in src/test/fuzz/utxo_snapshot.cpp for both “Release” and “Debug” configurations:

0< snip >
1  utxo_snapshot.cpp
2C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: Internal compiler error. [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
3C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: (compiler file 'D:\a\_work\1\s\src\vctools\Compiler\CxxFE\sl\p1\c\symbols.c', line 33772) [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
4C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001:  To work around this problem, try simplifying or changing the program near the locations listed above. [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
5C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: If possible please provide a repro here: https://developercommunity.visualstudio.com  [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
6C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: Please choose the Technical Support command on the Visual C++  [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
7C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001:  Help menu, or open the Technical Support help file for more information [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]

According to Microsoft docs:

If the file has a cxxfe …, it is probably a parser error.

The issue also occurs in recent CI jobs using the image version 20241113.3.0.

hebasto added the label Windows on Nov 17, 2024

hebasto added the label Upstream on Nov 17, 2024

hebasto commented at 10:48 am on November 17, 2024: member

cc @sipsorcery @davidgumberg @hodlinator

maflcko commented at 10:50 am on November 17, 2024: member

It would be good to report this upstream, as requested in the error message. Possibly with a minimal reproducer, if someone on Windows can reduce it further.

hebasto referenced this in commit b2d5361002 on Nov 17, 2024

sipsorcery commented at 5:00 pm on November 17, 2024: contributor

@hebasto I have never buil the fuzzer before. Is the error from the build here?

hebasto commented at 5:04 pm on November 17, 2024: member

@sipsorcery

I have never buil the fuzzer before. Is the error from the build here?

Just configure with -DBUILD_FUZZ_BINARY=ON and build the fuzz target.

maflcko commented at 8:16 am on November 18, 2024: member

According to #27930 (comment) MSVC is including Bitcoin Core in an internal test-suite, but I guess they are not compiling the fuzz tests? cc @CaseyCarter

CaseyCarter commented at 8:38 am on November 18, 2024: contributor

I see a -DBUILD_FUZZ_BINARY=ON in the setup, which suggests to me that we are at least building the fuzz tests. We are, however, at 0ca1d1b which seems to be about a month old. Our Real-World-Code projects don’t live at head, they jump forward periodically. If it’s a recent regression we may not have seen it yet.

hebasto commented at 2:36 pm on November 18, 2024: member

I see a -DBUILD_FUZZ_BINARY=ON in the setup, which suggests to me that we are at least building the fuzz tests. We are, however, at 0ca1d1b which seems to be about a month old. Our Real-World-Code projects don’t live at head, they jump forward periodically. If it’s a recent regression we may not have seen it yet.

 0> git rev-parse HEAD
 10ca1d1bf69ca364393e924cf41becfde1b68126c
 2> cmake -B build-static --preset vs2022-static --fresh -DVCPKG_MANIFEST_NO_DEFAULT_FEATURES=ON -DVCPKG_MANIFEST_FEATURES="wallet;tests" -DBUILD_GUI=OFF -DBUILD_FUZZ_BINARY=ON
 3> cmake --build build-static -t fuzz --config Release -j 8
 4MSBuild version 17.12.6+db5f6012c for .NET Framework
 5...
 6C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: Internal compiler error. [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
 7C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: (compiler file 'D:\a\_work\1\s\src\vctools\Compiler\CxxFE\sl\p1\c\symbols.c', line 33772) [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
 8C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001:  To work around this problem, try simplifying or changing the program near the locations listed above. [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
 9C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: If possible please provide a repro here: https://developercommunity.visualstudio.com  [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
10C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001: Please choose the Technical Support command on the Visual C++  [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]
11C:\Users\hebasto\source\repos\bitcoin\src\test\fuzz\utxo_snapshot.cpp(45,1): error C1001:  Help menu, or open the Technical Support help file for more information [C:\Users\hebasto\source\repos\bitcoin\build-static\src\test\fuzz\fuzz.vcxproj]

CaseyCarter commented at 0:01 am on November 19, 2024: contributor

I dug through the logs for yesterday’s nightly run, and the trunk compiler built utxo_snapshot.cpp just fine. I see: we onboarded bitcoin on 2024-10-22. It’s likely the 17.12 compilers have never seen it. Sorry about the regression; go ahead and file a report at https://developercommunity.visualstudio.com/cpp/report and we’ll see if we can get it serviced quickly.

fanquake referenced this in commit 746f93b4f0 on Nov 19, 2024

hebasto commented at 11:27 am on November 22, 2024: member

Sorry about the regression; go ahead and file a report at https://developercommunity.visualstudio.com/cpp/report and we’ll see if we can get it serviced quickly.

https://developercommunity.visualstudio.com/t/C1001:-Internal-compiler-error-in-the-Bi/10797559

petamas commented at 4:56 pm on January 13, 2025: none

@hebasto: Did you get a reply in the Developer Community issue? I’ve run into the same ICE at my workplace, but I don’t have access to the linked issue. (“We were unable to get this feedback item. It could be because you don’t have access to it or it doesn’t exist”) Thanks!

hebasto commented at 9:18 am on January 14, 2025: member

@hebasto: Did you get a reply in the Developer Community issue?

Not really. I only received an email confirming the creation of a new issue, along with the link I posted above.

I’ve run into the same ICE at my workplace, but I don’t have access to the linked issue. (“We were unable to get this feedback item. It could be because you don’t have access to it or it doesn’t exist”) Thanks!

It’s strange, but the link I received doesn’t work for me either.

cc @CaseyCarter

maflcko commented at 1:30 pm on January 15, 2025: member

Could create a new issue? Maybe it hit an automated filter?

hebasto commented at 11:40 am on January 17, 2025: member

Could create a new issue? Maybe it hit an automated filter?

I’ve just created a new issue. The “View full details” link I received via email shows:

We were unable to get this feedback item. It could be because you don’t have access to it or it doesn’t exist

Error details: [“Request failed with status code 404”]

janus referenced this in commit 05c3e6223e on Jan 19, 2025

hebasto commented at 12:18 pm on January 19, 2025: member

I’ve managed to provide a proper repro and submitted another issue: https://developercommunity.visualstudio.com/t/C1001:-Internal-compiler-error-with-st/10830350.

maflcko commented at 9:05 am on January 20, 2025: member

I’ve managed to provide a proper repro and submitted another issue: https://developercommunity.visualstudio.com/t/C1001:-Internal-compiler-error-with-st/10830350.

Thanks! Looks like it is already pending a release.

MSVC 17.12.0 internal compiler error #31303