https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
Andreas Krebbel changed:
What|Removed |Added
Status|RESOLVED|CLOSED
--- Comment #35 from Andreas
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
Andreas Krebbel changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #33 from Andreas Krebbel ---
Author: krebbel
Date: Fri Dec 2 08:30:16 2016
New Revision: 243160
URL: https://gcc.gnu.org/viewcvs?rev=243160=gcc=rev
Log:
PR target/77822: S390: Validate argument range of {zero,sign}_extract.
With
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #32 from Andreas Krebbel ---
Author: krebbel
Date: Fri Dec 2 08:26:19 2016
New Revision: 243159
URL: https://gcc.gnu.org/viewcvs?rev=243159=gcc=rev
Log:
PR target/77822: Add helper macro EXTRACT_ARGS_IN_RANGE to system.h.
The
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #31 from Dominik Vogt ---
No more backports, but the S390 fix for trunk is still in the queue. After it
gets the bug can be resolved.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #30 from ktkachov at gcc dot gnu.org ---
Fixed on aarch64 for 6.3 as well. There's nothing more to do for aarch64 here.
Dominik, do you intend to do any backports of the s390 patches? Or can we close
this?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #29 from ktkachov at gcc dot gnu.org ---
Author: ktkachov
Date: Tue Nov 22 16:55:16 2016
New Revision: 242709
URL: https://gcc.gnu.org/viewcvs?rev=242709=gcc=rev
Log:
[AArch64] Fix PR target/77822: Use tighter predicates for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #28 from Andreas Krebbel ---
Author: krebbel
Date: Fri Nov 11 10:37:53 2016
New Revision: 242067
URL: https://gcc.gnu.org/viewcvs?rev=242067=gcc=rev
Log:
S/390: Add PR77822 testcase.
For real this time.
2016-11-11 Dominik Vogt
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #27 from Andreas Krebbel ---
Author: krebbel
Date: Fri Nov 11 08:48:29 2016
New Revision: 242066
URL: https://gcc.gnu.org/viewcvs?rev=242066=gcc=rev
Log:
PR77822: S/390: Add range checks for zero_extract operands.
Make sure the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #26 from Dominik Vogt ---
Patch for s390[x], gcc-6:
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg00745.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #25 from Dominik Vogt ---
I see.
This test verifies that a negative "pos" is indeed rejected:
--
#include
int g;
void foo(int64_t b)
{
if (b >> 65 & 1)
g = b;
}
--
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #24 from ktkachov at gcc dot gnu.org ---
(In reply to Dominik Vogt from comment #23)
> Regarding the ARM patch:
>
> + {
> +if (!IN_RANGE (INTVAL (operands[2]) + INTVAL (operands[3]),
> +1, GET_MODE_BITSIZE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #23 from Dominik Vogt ---
Regarding the ARM patch:
+ {
+if (!IN_RANGE (INTVAL (operands[2]) + INTVAL (operands[3]),
+ 1, GET_MODE_BITSIZE (DImode) - 1))
+ FAIL;
+ }
Isn't this patch too simple? On s390x
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
Dominik Vogt changed:
What|Removed |Added
CC||vogt at linux dot vnet.ibm.com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #21 from Segher Boessenkool ---
Ah, so combine isn't really doing anything wrong here -- for the "plain"
shift it already only refuses it because the target does not allow it.
Some targets *do* allow shifting by amounts more than a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #20 from ktkachov at gcc dot gnu.org ---
One of the RTL sequences before combine showing the problem is:
(insn 94 93 95 18 (set (reg:DI 143)
(const_int 160 [0xa0])) "bad.cpp":19 50 {*movdi_aarch64}
(nil))
(insn 95 94 96
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Summary|[6/7 Regression] arm64 |[6 Regression] arm64 Error:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #11 from Jordi Mallach ---
Created attachment 39747
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39747=edit
ii file
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #12 from Jordi Mallach ---
Created attachment 39748
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39748=edit
s file
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #10 from Jordi Mallach ---
OK, to avoid the .gch includes, build with PRECOMPILE=0 REGENIE=1. I'm
attaching the result.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822
--- Comment #9 from Andrew Pinski ---
do the following to remove the PCH:
rm -rF ./obj/Release/emu.h.gch
and then rebuild the preprocessed source.
21 matches
Mail list logo