On Wed, Sep 24, 2014 at 9:08 PM, Michael Collison
<michael.collison@linaro.org> wrote:
Testcase attaached. Reproduce with "g++ -c -O2 -fPIE". URL for more detailed
description:
https://bugs.linaro.org/show_bug.cgi?id=331
It would be a good idea to get a reduced testcase that you could add
to the GCC testsuite so this won't show up again.
Thanks,
Andrew Pinski
On 09/24/2014 09:01 PM, Andrew Pinski wrote:
On Wed, Sep 24, 2014 at 8:45 PM, Michael Collison
<michael.collison@linaro.org> wrote:
On certain patterns in atomics.md the constraint 'n' is used in
combination
with the predicate atomic_op_operand. The constraint is too general and
allows constants that are disallowed by the predicate. This causes an ICE
In
final_scan_insn when the insn cannot be split because the constraint and
predicate do not match.
Tested on aarch64-none-elf, aarch64-linux-gnu. Additionally the
originally
reporter of the bug, (doko@ubuntu.com), applied the patch and
successfully
bootstrapped and tested with no regressions.
Testcase?
2014-09-23 Michael Collison <michael.collison@linaro.org>
* config/aarch64/iterators.md (lconst_atomic): New mode attribute to
support constraints for CONST_INT in atomic operations.
* config/aarch64/atomics.md
(atomic_<atomic_optab><mode>): Use lconst_atomic constraint.
(atomic_nand<mode>): Likewise.
(atomic_fetch_<atomic_optab><mode>): Likewise.
(atomic_fetch_nand<mode>): Likewise.
(atomic_<atomic_optab>_fetch<mode>): Likewise.
(atomic_nand_fetch<mode>): Likewise.
--
Michael Collison
Linaro Toolchain Working Group
michael.collison@linaro.org
--
Michael Collison
Linaro Toolchain Working Group
michael.collison@linaro.org