Commit 0b661358 by Palmer Dabbelt Committed by Palmer Dabbelt

RISC-V: Add Sign/Zero extend patterns for PIC loads

Loads on RISC-V are sign-extending by default, but we weren't telling
GCC this in our PIC load patterns.  This corrects the problem, and adds
a zero-extending pattern as well.

gcc/ChangeLog

2017-10-25  Palmer Dabbelt  <palmer@dabbelt.com>

       * config/riscv/riscv.md (ZERO_EXTEND_LOAD): Define.
       * config/riscv/pic.md (local_pic_load): Rename to local_pic_load_s,
       mark as a sign-extending load.
       (local_pic_load_u): Define.

From-SVN: r254092
parent 4273ea23
2017-10-25 Palmer Dabbelt <palmer@dabbelt.com>
* config/riscv/riscv.md (ZERO_EXTEND_LOAD): Define.
* config/riscv/pic.md (local_pic_load): Rename to local_pic_load_s,
mark as a sign-extending load.
(local_pic_load_u): Define.
2017-10-25 Eric Botcazou <ebotcazou@adacore.com> 2017-10-25 Eric Botcazou <ebotcazou@adacore.com>
PR middle-end/82062 PR middle-end/82062
...@@ -22,13 +22,20 @@ ...@@ -22,13 +22,20 @@
;; Simplify PIC loads to static variables. ;; Simplify PIC loads to static variables.
;; These should go away once we figure out how to emit auipc discretely. ;; These should go away once we figure out how to emit auipc discretely.
(define_insn "*local_pic_load<mode>" (define_insn "*local_pic_load_s<mode>"
[(set (match_operand:ANYI 0 "register_operand" "=r") [(set (match_operand:ANYI 0 "register_operand" "=r")
(mem:ANYI (match_operand 1 "absolute_symbolic_operand" "")))] (sign_extend:ANYI (mem:ANYI (match_operand 1 "absolute_symbolic_operand" ""))))]
"USE_LOAD_ADDRESS_MACRO (operands[1])" "USE_LOAD_ADDRESS_MACRO (operands[1])"
"<load>\t%0,%1" "<load>\t%0,%1"
[(set (attr "length") (const_int 8))]) [(set (attr "length") (const_int 8))])
(define_insn "*local_pic_load_u<mode>"
[(set (match_operand:ZERO_EXTEND_LOAD 0 "register_operand" "=r")
(zero_extend:ZERO_EXTEND_LOAD (mem:ZERO_EXTEND_LOAD (match_operand 1 "absolute_symbolic_operand" ""))))]
"USE_LOAD_ADDRESS_MACRO (operands[1])"
"<load>u\t%0,%1"
[(set (attr "length") (const_int 8))])
(define_insn "*local_pic_load<mode>" (define_insn "*local_pic_load<mode>"
[(set (match_operand:ANYF 0 "register_operand" "=f") [(set (match_operand:ANYF 0 "register_operand" "=f")
(mem:ANYF (match_operand 1 "absolute_symbolic_operand" ""))) (mem:ANYF (match_operand 1 "absolute_symbolic_operand" "")))
......
...@@ -259,6 +259,9 @@ ...@@ -259,6 +259,9 @@
;; Iterator for QImode extension patterns. ;; Iterator for QImode extension patterns.
(define_mode_iterator SUPERQI [HI SI (DI "TARGET_64BIT")]) (define_mode_iterator SUPERQI [HI SI (DI "TARGET_64BIT")])
;; Iterator for extending loads.
(define_mode_iterator ZERO_EXTEND_LOAD [QI HI (SI "TARGET_64BIT")])
;; Iterator for hardware integer modes narrower than XLEN. ;; Iterator for hardware integer modes narrower than XLEN.
(define_mode_iterator SUBX [QI HI (SI "TARGET_64BIT")]) (define_mode_iterator SUBX [QI HI (SI "TARGET_64BIT")])
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment