https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
Jim Wilson changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
--- Comment #8 from Jim Wilson ---
Author: wilson
Date: Mon Apr 2 22:37:21 2018
New Revision: 259019
URL: https://gcc.gnu.org/viewcvs?rev=259019=gcc=rev
Log:
RISC-V: Fix for combine bug with shift and AND operations.
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
--- Comment #7 from Kito Cheng ---
Hi Jim:
Thanks your update, we'll test that again, btw, we can also test on x86-64 and
aarch64 if you have no time to test other target.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
Jim Wilson changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |wilson at gcc dot
gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
--- Comment #5 from Jim Wilson ---
This is what the original patch I wrote looked like. But then I tried to
optimize it a bit to eliminate the redundant nonzero_bits call, and that is
where I screwed up. Then I got overwhelmed by work and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
--- Comment #4 from Kito Cheng ---
Hi Jim:
We've written a patch which is based on your patch for gcc 7, and we've tested
with GCC testsuite and several testsuite.
diff --git a/gcc/combine.c b/gcc/combine.c
index ff250aa..033387d 100644
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
--- Comment #3 from Jim Wilson ---
I screwed up the logic and need to redo the patch. The basic idea is sound
though.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
--- Comment #2 from Jim Wilson ---
Created attachment 43553
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43553=edit
untested patch that works on testcase
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84660
Jim Wilson changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
