Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

<ranges>: ICE with std::views::join_with #17

Closed
2 tasks done
cpplearner opened this issue Jul 27, 2022 · 4 comments
Closed
2 tasks done

<ranges>: ICE with std::views::join_with #17

cpplearner opened this issue Jul 27, 2022 · 4 comments
Labels
fix-released Compiler fix has shipped ICE-on-valid 7 points

Comments

@cpplearner
Copy link

Basic Info

  • This bug is:
    ICE-on-valid

Command-Line Test Case

My test case that works with classic includes but fails with named modules:

D:\bug-bash>type test.cpp
#ifdef USE_CLASSIC_INCLUDES
    #include <array>
    #include <ranges>
#else
    import std;
#endif

int main() {
    std::array<std::array<char, 10>, 10>{} | std::views::join_with(' ');
}
D:\bug-bash>cl /EHsc /nologo /W4 /std:c++latest /MTd /c /FoNUL /DUSE_CLASSIC_INCLUDES test.cpp
test.cpp

D:\bug-bash>cl /EHsc /nologo /W4 /std:c++latest /MTd /c /FoNUL test.cpp
test.cpp
D:\bug-bash\out\x64\out\inc\ranges(3883): fatal error C1001: Internal compiler error.
(compiler file 'msc1.cpp', line 1576)
 To work around this problem, try simplifying or changing the program near the locations listed above.
If possible please provide a repro here: https://developercommunity.visualstudio.com
Please choose the Technical Support command on the Visual C++
 Help menu, or open the Technical Support help file for more information
INTERNAL COMPILER ERROR in 'D:\Program Files\Microsoft Visual Studio\2022\Preview\VC\Tools\MSVC\14.33.31627\bin\HostX64\x64\cl.exe'
    Please choose the Technical Support command on the Visual C++
    Help menu, or open the Technical Support help file for more information

Additional Context

None.

@StephanTLavavej
Copy link
Owner

Thanks for the excellent test case! I'll report this to the compiler team later today.

@StephanTLavavej
Copy link
Owner

Confirmed, still repros with 17.3 Preview 5. Reported as internal VSO-1582910 "Standard Library Modules: ICE with std::views::join_with".

@StephanTLavavej
Copy link
Owner

A compiler fix for this ICE is undergoing review!

@StephanTLavavej StephanTLavavej added fix-pending-release Compiler fix will ship soon-ish and removed reported Reported to the compiler team labels Aug 5, 2022
@StephanTLavavej
Copy link
Owner

Compiler fix will ship in 17.4 Preview 3.

@StephanTLavavej StephanTLavavej added fix-released Compiler fix has shipped and removed fix-pending-release Compiler fix will ship soon-ish labels Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix-released Compiler fix has shipped ICE-on-valid 7 points
Projects
None yet
Development

No branches or pull requests

2 participants