-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Segfault during i686 windows bootstrap #18162
Comments
doesn't happen on gcc 4.9, does happen with 5.4.0 and 6.1.0 |
Looks like a miscompile of some sort to me.
|
Investigated and and filed upstream as https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77333. |
Assuming this is reproducible by cross-compiling from Linux it seems to be fixed with 6.2.
|
I certainly saw this in a cross compile with 6.2. I didn't try LLVM 3.9, so it's possibly not triggering the gcc bug in the same way that 3.7 was. |
I can confirm that this doesn't happen when using LLVM 3.9 with i686-w64-mingw32-gcc 5.4 from cygwin, but it does happen with LLVM 3.7 and the same compiler version. So something that changed in LLVM is no longer triggering the GCC bug, but that doesn't inspire a lot of confidence that it won't come back at some point. And we need to keep the buildbots capable of building release-0.5 which won't be upgrading LLVM, so we need to have a compiler version installed there that can successfully build with LLVM 3.7 for some time. |
Apparently the commit that caused this to stop happening on the llvm side was llvm-mirror/llvm@225dd82d634ca277 - which looks pretty small and not hugely consequential? |
Yeah, that gcc bug should still be fixed. |
Any ideas how we get a gcc dev to triage it? We could attempt to run |
Maybe ping the developer whose commit caused the regression? |
Fixed upstream in gcc 7.1, 6.4, and 5.5 (if there will ever be a 5.5) - the cygwin cross compilers are at 5.4 and they're carrying a patch that isn't quite what got applied in the end, but seemed to fix the issue (who knows if any other side effects though). |
May or may not be related to #18123, but I'm still seeing this with binutils 2.27, so it's not fixed yet by that version:
The text was updated successfully, but these errors were encountered: