[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Jonathan Wakely  changed:

   What|Removed |Added

 CC||tprince at computer dot org

--- Comment #34 from Jonathan Wakely  ---
*** Bug 69266 has been marked as a duplicate of this bug. ***

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #33 from Nick Clifton  ---
The patch did receive approval in the end:

  https://gcc.gnu.org/ml/gcc-patches/2016-01/msg02074.html

phew!

As for 68601 - I guess that we can leave it as RESOLVED for now, and see if the
problem resurfaces at a later date.  (Fingers crossed - this will not happen).

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rai...@emrich-ebersheim.de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #25 from Rainer Emrich  ---
I'm testing rev. 232815, proposed patch applied, on x86_64-w64-mingw32 atm.
Will take some time.

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #26 from Nick Clifton  ---
Hi Roger,

> I've tried the patch (applied to 232400 as trunk seems to have other
> problems on cygwin) and the build now completes successfully.

Including libstdc++-v3 ?

> Additionally, the test case no longer crashes.

Hooray.

Now all we need is for Rainer to confirm that the patch works for mingw32 and
we are good to go.

Oh actually - Roger - could you check one more thing for me please ?  Does the
patched compiler compile the test case in PR 68601 correctly ?

Cheers
  Nick

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rai...@emrich-ebersheim.de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #27 from Rainer Emrich  ---
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256

Am 26.01.2016 um 10:30 schrieb nickc at gcc dot gnu.org:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655
> 
> --- Comment #26 from Nick Clifton  --- Hi
> Roger,
> 
>> I've tried the patch (applied to 232400 as trunk seems to have
>> other problems on cygwin) and the build now completes
>> successfully.
> 
> Including libstdc++-v3 ?
I did a full bootstrap including all working languages for
x86_64-w64-mingw32 and a full testsuite run. You may find the results
at gcc-testresults:
https://gcc.gnu.org/ml/gcc-testresults/2016-01/msg02541.html

> 
>> Additionally, the test case no longer crashes.
The test case does and did PASS.

> 
> Hooray.
> 
> Now all we need is for Rainer to confirm that the patch works for
> mingw32 and we are good to go.

Yeah, we good to go, congratulations, thank you.

Cheers

Rainer

-BEGIN PGP SIGNATURE-
Version: GnuPG v2

iQEcBAEBCAAGBQJWp3V1AAoJEB3HOsWs+KJb1KkH/ApsmUJNHLIRIsdA9ayLgW2J
qoIFobCZSJWNzEPMv/a92zhC/9JJqh7HqkrCdb46Dmlm3FGZjIuhzfyreeI+xyNj
PNuXJoXV2US7s5ywhE2DbiOmsEkboe/vYBTUvsA7179YmZdBm3718tJU0U/IBeim
AB3rmtHIs0F8Ousho8kfwiF8fGY4KA0pF9i7PT1gyXpFyNTbljXom8nUhP/gZDJU
2V6/GPpu0kuP/0KLydBSDGUZYuujs/HZ7BMBfJRmpWyy4YDcSFikaroTSr3XXjad
oBZySKt/DuuRvnEhrOll4xVJTg6bfQhNhf0hS3L/BRG/IdhELrJvnpT/8WjtFL0=
=pSyU
-END PGP SIGNATURE-

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #28 from Nick Clifton  ---
Author: nickc
Date: Tue Jan 26 14:02:11 2016
New Revision: 232828

URL: https://gcc.gnu.org/viewcvs?rev=232828=gcc=rev
Log:
PR target/66655
* config/i386/winnt.c (i386_pe_binds_local_p): If a function has
been marked as DECL_ONE_ONLY but we do not the means to make it
so, then do not allow it to bind locally.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/winnt.c

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Nick Clifton  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #29 from Nick Clifton  ---
New patch applied.  This time it works, and no-one will have to be nailed to
anything...

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[amonakov at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #30 from Alexander Monakov  ---
Nick, can you please post the patch to gcc-patches too, to avoid confusing
future people who wouldn't be able to find the explanation of the patch in the
archives?

(did you get approval for this patch offline?)

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at redhat dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #31 from Nick Clifton  ---
Hi Alexander,

> Nick, can you please post the patch to gcc-patches too, to avoid confusing
> future people who wouldn't be able to find the explanation of the patch in the
> archives?
> (did you get approval for this patch offline?)

Oh foo - no I did not.  I will post to gcc-patches now.  If approval is
forthcoming quickly then I will leave things as they are.  Otherwise I will
revert my patch and wait for proper authorisation.  Sorry about mucking up.

Cheers
   Nick

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rogero at howzatt dot demon.co.uk]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #32 from Roger Orr  ---
(In reply to Nick Clifton from comment #26)
> Hi Roger,
> 
> > I've tried the patch (applied to 232400 as trunk seems to have other
> > problems on cygwin) and the build now completes successfully.
> 
> Including libstdc++-v3 ?

Yes, the build included libstdc++-v3

...

> Oh actually - Roger - could you check one more thing for me please ?  Does
> the patched compiler compile the test case in PR 68601 correctly ?

I'm not sure how to compile the Instructions.ii -- a naive
$ /usr/share/gcc-trunk/bin/g++ -O2 -fno-inline Instructions.ii
errors with:

In file included from
c:\msys64\home\administrator\julia\usr\x86_64-w64-mingw32\sys-root\mingw\lib\gcc\x86_64-w64-mingw32\5.2.0\include\c++\utility:75:0,
 from
c:\msys64\home\administrator\julia\usr\x86_64-w64-mingw32\sys-root\mingw\lib\gcc\x86_64-w64-mingw32\5.2.0\include\c++\algorithm:60,
 from
C:/msys64/home/Administrator/julia/deps/srccache/llvm-svn/include/llvm/ADT/StringRef.h:14,
 from
C:/msys64/home/Administrator/julia/deps/srccache/llvm-svn/include/llvm/ADT/StringMap.h:17,
 from
C:/msys64/home/Administrator/julia/deps/srccache/llvm-svn/include/llvm/Support/Host.h:17,
 from
C:/msys64/home/Administrator/julia/deps/srccache/llvm-svn/include/llvm/ADT/Hashing.h:49,
 from
C:/msys64/home/Administrator/julia/deps/srccache/llvm-svn/include/llvm/ADT/ArrayRef.h:13,
 from
C:/msys64/home/Administrator/julia/deps/srccache/llvm-svn/include/llvm/IR/Instructions.h:19,
 from
C:/msys64/home/Administrator/julia/deps/srccache/llvm-svn/lib/IR/Instructions.cpp:16:
c:\msys64\home\administrator\julia\usr\x86_64-w64-mingw32\sys-root\mingw\lib\gcc\x86_64-w64-mingw32\5.2.0\include\c++\initializer_list:47:11:
fatal error: definition of std::initializer_list does not match #include

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rogero at howzatt dot demon.co.uk]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #24 from Roger Orr  ---
Thanks Nick.

I've tried the patch (applied to 232400 as trunk seems to have other problems
on cygwin) and the build now completes successfully.

Additionally, the test case no longer crashes.

$ /usr/share/gcc-trunk/bin/g++ -O2 -fno-inline pr66655.C pr66655_1.cc
$ ./a.exe
$ /usr/share/gcc-trunk/bin/g++ -O2 -fno-inline pr66655_1.cc pr66655.C
$ ./a.exe

$ /usr/share/gcc-trunk/bin/g++ -v
Using built-in specs.
COLLECT_GCC=/usr/share/gcc-trunk/bin/g++
COLLECT_LTO_WRAPPER=/usr/share/gcc-trunk/libexec/gcc/x86_64-unknown-cygwin/6.0.0/lto-wrapper.exe
Target: x86_64-unknown-cygwin
Configured with: ../gcc-trunk-232400/configure --enable-languages=c,c++
--prefix=/usr/share/gcc-trunk
Thread model: single
gcc version 6.0.0 20160115 (experimental) (GCC)

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Nick Clifton  changed:

   What|Removed |Added

  Attachment #37064|0   |1
is obsolete||

--- Comment #23 from Nick Clifton  ---
Created attachment 37465
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37465=edit
New proposed patch

Hi Guys,

  This is a new patch that I am currently testing locally.  If anyone else
would like to give it a go and let me if it really does work (and not break
building libstdc++-v3) I would be very grateful.

Cheers
  Nick

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rogero at howzatt dot demon.co.uk]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #18 from Roger Orr  ---
What is the correct way forward with this issue?
Should we reopen this issue, or open a fresh one? (I'm not familiar enough with
the gcc use of bugzilla to know.)

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Nick Clifton  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
   Last reconfirmed||2016-01-25
 Resolution|FIXED   |---
 Ever confirmed|0   |1

--- Comment #19 from Nick Clifton  ---
The patch has been shown to break libstdc++-v3 builds, so back to the drawing
board...

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #20 from Nick Clifton  ---
Author: nickc
Date: Mon Jan 25 08:57:02 2016
New Revision: 232785

URL: https://gcc.gnu.org/viewcvs?rev=232785=gcc=rev
Log:
PR target/66655
* config/i386/cygming.h (MAKE_DECL_ONE_ONLY): Remove definition.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/cygming.h

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #21 from Nick Clifton  ---
Hi Roger,

> What is the correct way forward with this issue?

Sorry - my bad - I meant to get to this last Friday, but was snowed under.

We need to reopen the bug (done), remove the broken patch from the sources
(done) and reinvestigate the problem (not done),

Cheers
  Nick

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Jakub Jelinek  changed:

   What|Removed |Added

 CC||rai...@emrich-ebersheim.de

--- Comment #22 from Jakub Jelinek  ---
*** Bug 69440 has been marked as a duplicate of this bug. ***

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at gcc dot gnu.org

--- Comment #15 from Nick Clifton  ---
Hi Roger, Hi Tony,

I have been trying, unsuccessfully, to build a complete cygwin toolchain, but I
was able to reproduce the reported failure.

I did try out the patch below, which implements Tony's idea of restricting my
original patch to just Mingw targets, but I was still unable to build a full
toolchain (for other reasons).  Roger - perhaps you would be able to test out
the patch in your build environment for me ?

Cheers
  Nick

Index: gcc/config/i386/cygwin.h
===
--- gcc/config/i386/cygwin.h(revision 232666)
+++ gcc/config/i386/cygwin.h(working copy)
@@ -157,3 +157,5 @@
 /* We should find a way to not have to update this manually.  */
 #define LIBGCJ_SONAME "cyggcj" /*LIBGCC_EH_EXTN*/ "-16.dll"

+/* Cygwin does not support weak symbols.  */
+#undef MAKE_DECL_ONE_ONLY

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rogero at howzatt dot demon.co.uk]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #17 from Roger Orr  ---
As you say, there seems to be another, unrelated, problem with the current
trunk and cygwin.

However, I have now successfully built gcc version 232300 under cygwin with
this patch. 

Unfortunately, if I try to compile and execute the test program:

$ g++ -O2 -fno-inline pr66655.C pr66655_1.cc
$ ./a.exe

it produces a segmentation fault, just as reported with the mingw build.

So it does seem that cygwin needs the problem fixed, just like mingw, but that
the orginal fix has other, unfortunate, consequences.

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[fanael4 at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Fanael  changed:

   What|Removed |Added

 CC||fanael4 at gmail dot com

--- Comment #16 from Fanael  ---
The same linker errors happen when building a mingw toolchain.

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rogero at howzatt dot demon.co.uk]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #14 from Roger Orr  ---
I don't know I'm afraid: I'm a very occasional mingw user and I haven't
(yet...) tried building gcc on mingw...

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rogero at howzatt dot demon.co.uk]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Roger Orr  changed:

   What|Removed |Added

 CC||rogero at howzatt dot 
demon.co.uk

--- Comment #12 from Roger Orr  ---
Alas, revision 232071 appears to break the cygwin full build of gcc.

$ cd /cygdrive/c/projects/gcc/gcctrunk
$ svn update -r 232071
$ ./contrib/download_prerequisites
$ mkdir ../build
$ cd ../build
$ ../gcctrunk/configure --enable-languages=c,c++ --prefix=/usr/share/gcctrunk
$ make -j4 2>&1 | tee /var/tmp/build.log
...
libtool: link:  /cygdrive/c/projects/gcc/build/./gcc/xgcc -shared-libgcc
-B/cygdrive/c/projects/gcc/build/./gcc -nostdinc++
-L/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/src
-L/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/src/.libs
-L/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/libsupc++/.libs
-B/usr/share/gcctrunk/x86_64-unknown-cygwin/bin/
-B/usr/share/gcctrunk/x86_64-unknown-cygwin/lib/ -isystem
/usr/share/gcctrunk/x86_64-unknown-cygwin/include -isystem
/usr/share/gcctrunk/x86_64-unknown-cygwin/sys-include-shared -nostdlib
/cygdrive/c/projects/gcc/build/./gcc/crtbeginS.o  .libs/compatibility.o
.libs/compatibility-debug_list.o .libs/compatibility-debug_list-2.o
.libs/compatibility-c++0x.o .libs/compatibility-atomic-c++0x.o
.libs/compatibility-thread-c++0x.o .libs/compatibility-chrono.o
.libs/compatibility-condvar.o  -Wl,--whole-archive
../libsupc++/.libs/libsupc++convenience.a
../src/c++98/.libs/libc++98convenience.a
../src/c++11/.libs/libc++11convenience.a -Wl,--no-whole-archive 
-L/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/libsupc++/.libs
-L/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/src
-L/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/src/.libs
-L/usr/lib/w32api -L/cygdrive/c/projects/gcc/build/./gcc -L/lib/../lib
-L/usr/lib/../lib -lgcc_s -lgcc -lcygwin -ladvapi32 -lshell32 -luser32
-lkernel32 -lgcc_s -lgcc /cygdrive/c/projects/gcc/build/./gcc/crtend.o  -Wl,-O1
-Wl,--gc-sections -Wl,--version-script=libstdc++-symbols.ver   -o
.libs/cygstdc++-6.dll -Wl,--enable-auto-image-base -Xlinker --out-implib
-Xlinker .libs/libstdc++.dll.a
.libs/compatibility.o: In function `std::basic_istream::ignore(long)':
/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/src/../../../../gcctrunk/libstdc++-v3/src/c++98/compatibility.cc:67:
undefined reference to `std::basic_istream::sentry::sentry(std::basic_istream&, bool)'

(many similar errors elided)...

../src/c++11/.libs/libc++11convenience.a(wstring-inst.o):/cygdrive/c/projects/gcc/gcctrunk/libstdc++-v3/include/bits/basic_string.h:196:
more undefined references to `void std::__cxx11::basic_string::_M_construct(wchar_t const*, wchar_t const*, std::forward_iterator_tag)' follow
collect2: error: ld returned 1 exit status
Makefile:606: recipe for target 'libstdc++.la' failed
make[6]: *** [libstdc++.la] Error 1
make[6]: Leaving directory
'/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/src'
Makefile:638: recipe for target 'all-recursive' failed
make[5]: *** [all-recursive] Error 1
make[5]: Leaving directory
'/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3/src'
Makefile:507: recipe for target 'all-recursive' failed
make[4]: *** [all-recursive] Error 1
make[4]: Leaving directory
'/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3'
Makefile:414: recipe for target 'all' failed
make[3]: *** [all] Error 2
make[3]: Leaving directory
'/cygdrive/c/projects/gcc/build/x86_64-unknown-cygwin/libstdc++-v3'
Makefile:16346: recipe for target 'all-stage1-target-libstdc++-v3' failed
make[2]: *** [all-stage1-target-libstdc++-v3] Error 2
make[2]: Leaving directory '/cygdrive/c/projects/gcc/build'
Makefile:21374: recipe for target 'stage1-bubble' failed
make[1]: *** [stage1-bubble] Error 2
make[1]: Leaving directory '/cygdrive/c/projects/gcc/build'
Makefile:916: recipe for target 'all' failed
make: *** [all] Error 2

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[tony at kelman dot net]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #13 from Tony Kelman  ---
Should this change be for ming-but-not-cyg then?

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at redhat dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #11 from Nick Clifton  ---
Hi Marek,

> You need to sign in with your @gcc.gnu.org address.

Doh!  Totally forgot about that.  Thanks!

Cheers
   Nick

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

--- Comment #7 from Nick Clifton  ---
Author: nickc
Date: Tue Jan  5 10:57:01 2016
New Revision: 232071

URL: https://gcc.gnu.org/viewcvs?rev=232071=gcc=rev
Log:
PR target/66655
* config/i386/cygming.h (MAKE_DECL_ONE_ONLY): Define to use weak
decls if weak support is available.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/cygming.h

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[tony at kelman dot net]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Tony Kelman  changed:

   What|Removed |Added

 CC||tony at kelman dot net

--- Comment #6 from Tony Kelman  ---
I applied Nick's patch to OpenSUSE's build of mingw64-gcc 5.3.0 here:
https://build.opensuse.org/package/show/home:kelman:branches:windows:mingw:win64/mingw64-gcc

And the resulting build did not exhibit the abort in the original test case,
whereas the unpatched build at
https://build.opensuse.org/package/show/windows:mingw:win64/mingw64-gcc does
reproduce the abort.

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[nickc at redhat dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

--- Comment #5 from Nick Clifton  ---
Created attachment 37064
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37064=edit
Define MAKE_DECL_ONE_ONLY for cygming targets

Hi Guys,

  I think that this patch should resolve the problem, although I do not have
the means to test it locally.

  The issue I believe is not that the ming32 definition of bind_local_p is
wrong, but rather than G++ thinks that it cannot make the decl weak even though
bind_local_p says that it should.  By defining MAKE_DECL_ONE_ONLY we allow G++
to make the decl weak and so the compilation works.  Well that is the theory at
least...

  If someone could test out this patch I would be very grateful.

Cheers
  Nick

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|5.3 |5.4

--- Comment #4 from Richard Biener  ---
GCC 5.3 is being released, adjusting target milestone.

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #3 from Jakub Jelinek  ---
This then requires somebody familiar with the PE-COFF symbol
visibility/overriding to adjust the mingw targetm.binds_local_p or arrange to
adjust decl_binds_to_current_def_p for mingw.

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[amonakov at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Alexander Monakov  changed:

   What|Removed |Added

 CC||kfischer at college dot 
harvard.ed
   ||u

--- Comment #2 from Alexander Monakov  ---
*** Bug 68601 has been marked as a duplicate of this bug. ***

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Richard Biener  changed:

   What|Removed |Added

   Priority|P3  |P2

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Target Milestone|5.2 |5.3

--- Comment #1 from Richard Biener rguenth at gcc dot gnu.org ---
GCC 5.2 is being released, adjusting target milestone to 5.3.

[Bug target/66655] [5/6 Regression] miscompilation due to ipa-ra on MinGW

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66655

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

Version|unknown |5.1.0
   Target Milestone|--- |5.2
Summary|[5.1 Regression]|[5/6 Regression]
   |miscompilation due to   |miscompilation due to
   |ipa-ra on MinGW |ipa-ra on MinGW