fbb ftbfs on arm64

Bug #1364979 reported by Matthias Klose
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro
Fix Released
Wishlist
fbb (Ubuntu)
Fix Released
Undecided
Unassigned
gcc-4.9 (Debian)
Fix Released
Unknown
gcc-4.9 (Ubuntu)
Fix Released
Undecided
Unassigned
Revision history for this message
In , Matthias Klose (doko) wrote :

Created attachment 143
preprocessed source

[forwarded from https://bugs.debian.org/758964]

seen with binutils trunk and current gcc-4.9 linaro:

$ gcc -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wall -Wstrict-prototypes -O2 -g -funsigned-char -D__LINUX__ -DPROTOTYPES -I../include -DUSE_NCURSES -D_FORTIFY_SOURCE=2 -c -o md5c.o md5c.c
/tmp/cc5W8BeX.s: Assembler messages:
/tmp/cc5W8BeX.s:609: Error: operand 1 should be an integer register -- `ldr v12,[x1]'
/tmp/cc5W8BeX.s:610: Error: invalid use of vector register at operand 1 -- `eor v12,x6,v12'

Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :

Also occurs when I tested with with FSF trunk gcc version 5.0.0 20140821 (experimental) (GCC) .

Changed in linaro:
importance: Unknown → Wishlist
Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :

Bug occurs in latest trunk as well

gcc version 5.0.0 20140903 (experimental) (GCC).

Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :

The following patch fixes this bug

Testing in progress

diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
index b5be79c..ed6e602 100644
--- a/gcc/config/aarch64/aarch64.md
+++ b/gcc/config/aarch64/aarch64.md
@@ -4026,7 +4026,7 @@
 })

 (define_insn "stack_protect_test_<mode>"
- [(set (match_operand:PTR 0 "register_operand")
+ [(set (match_operand:PTR 0 "register_operand" "=r")
        (unspec:PTR [(match_operand:PTR 1 "memory_operand" "m")
                     (match_operand:PTR 2 "memory_operand" "m")]
         UNSPEC_SP_TEST))

Changed in gcc-4.9 (Debian):
status: Unknown → New
Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :

Tested the Patch and sent it to FSF mailing list for review. Once approved I will backport it to GCC 4.9 linaro branch.

 ref: https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00317.html

Revision history for this message
In , Ryan S. Arnold (ryan-arnold) wrote :
Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :

Patch approved. Need to upstream and back port to linaro 4.9 and FSF 4.9 branches.

Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :
Changed in gcc-4.9 (Debian):
status: New → Fix Released
Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :
Changed in linaro:
status: Unknown → Fix Released
Changed in gcc-4.9 (Debian):
status: Fix Released → New
Revision history for this message
In , Matthias Klose (doko) wrote :

there still seems to be an issue, as reported in https://bugs.debian.org/758964

Revision history for this message
In , Matthias Klose (doko) wrote :

Created attachment 162
preprocessed source

Revision history for this message
In , Kumar Venkataramanan (venkataramanan-kumar) wrote :

There is a separate bug report opened please see
https://bugs.linaro.org/show_bug.cgi?id=667

I am closing this bug.

Changed in gcc-4.9 (Debian):
status: New → Fix Released
Revision history for this message
Matthias Klose (doko) wrote :

fixed for 14.10

Changed in gcc-4.9 (Ubuntu):
status: New → Fix Released
Changed in fbb (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.