[Bug rtl-optimization/64916] ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64916 --- Comment #8 from Jeffrey A. Law law at redhat dot com --- Alex, you're right. It's just an artifact which patch I pulled out of my todo list. Your patches are effectively doing the same thing. If you'd like I can add you to the ChangeLog entry for the fix.
[Bug rtl-optimization/64916] ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64916 --- Comment #9 from Alex Velenko Alex.Velenko at arm dot com --- Jeff, thank you for noticing this. I don't mind this time, I just hope this would not happen next time.
[Bug rtl-optimization/64916] ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64916 --- Comment #7 from Alex Velenko Alex.Velenko at arm dot com --- Just to note, my fix was earlier, yet none replied: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg00441.html
[Bug rtl-optimization/64916] ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64916 --- Comment #5 from Jeffrey A. Law law at gcc dot gnu.org --- Author: law Date: Tue Apr 21 05:23:08 2015 New Revision: 56 URL: https://gcc.gnu.org/viewcvs?rev=56root=gccview=rev Log: PR rtl-optimization/64916 * cfgcleanup.c (values_equal_p): New function. (can_replace_by): Use it. Modified: trunk/gcc/ChangeLog trunk/gcc/cfgcleanup.c
[Bug rtl-optimization/64916] ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64916 Jeffrey A. Law law at redhat dot com changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #6 from Jeffrey A. Law law at redhat dot com --- Fixed by Shiva's patch on the trunk.
[Bug rtl-optimization/64916] ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64916 Ramana Radhakrishnan ramana at gcc dot gnu.org changed: What|Removed |Added Target Milestone|5.0 |--- --- Comment #4 from Ramana Radhakrishnan ramana at gcc dot gnu.org --- Removing the target milestone given that we've agreed it's not a regression.
[Bug rtl-optimization/64916] ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64916 Jeffrey A. Law law at redhat dot com changed: What|Removed |Added Summary|[5 regression] ira.c|ira.c update_equiv_regs |update_equiv_regs patch |patch causes |causes |gcc/testsuite/gcc.target/ar |gcc/testsuite/gcc.target/ar |m/pr43920-2.c regression |m/pr43920-2.c regression| --- Comment #3 from Jeffrey A. Law law at redhat dot com --- Removing the regression marker. This testcase depended in incorrect behaviour of update_equiv_regs. So it's clearly a missed optimization, it does not qualify as a regression IMO. In particular note that r110 can have two values (0 or -1) thus creating a REG_EQUIV note for r110 anywhere is wrong.
