Extract the hardcoded values for the minimum PC-relative displacements
into named constants and document them.

gcc/ChangeLog:

        * config/aarch64/aarch64.md (BRANCH_LEN_P_128MiB): New constant.
        (BRANCH_LEN_N_128MiB): Likewise.
        (BRANCH_LEN_P_1MiB): Likewise.
        (BRANCH_LEN_N_1MiB): Likewise.
        (BRANCH_LEN_P_32KiB): Likewise.
        (BRANCH_LEN_N_32KiB): Likewise.
---
 gcc/config/aarch64/aarch64.md | 64 ++++++++++++++++++++++++++---------
 1 file changed, 48 insertions(+), 16 deletions(-)

diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
index d362a4c6f24..fffa97a1ef1 100644
--- a/gcc/config/aarch64/aarch64.md
+++ b/gcc/config/aarch64/aarch64.md
@@ -704,7 +704,23 @@ (define_insn "jump"
   [(set_attr "type" "branch")]
 )
 
+;; Maximum PC-relative positive/negative displacements for various branching
+;; instructions.
+(define_constants
+  [
+    ;; +/- 128MiB.  Used by B, BL.
+    (BRANCH_LEN_P_128MiB  134217724)
+    (BRANCH_LEN_N_128MiB -134217728)
+
+    ;; +/- 1MiB.  Used by B.<cond>, CBZ, CBNZ.
+    (BRANCH_LEN_P_1MiB  1048572)
+    (BRANCH_LEN_N_1MiB -1048576)
 
+    ;; +/- 32KiB.  Used by TBZ, TBNZ.
+    (BRANCH_LEN_P_32KiB  32764)
+    (BRANCH_LEN_N_32KiB -32768)
+  ]
+)
 
 ;; -------------------------------------------------------------------
 ;; Conditional jumps
@@ -768,13 +784,17 @@ (define_insn "aarch64_bcond"
   }
   [(set_attr "type" "branch")
    (set (attr "length")
-       (if_then_else (and (ge (minus (match_dup 2) (pc)) (const_int -1048576))
-                          (lt (minus (match_dup 2) (pc)) (const_int 1048572)))
+       (if_then_else (and (ge (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_N_1MiB))
+                          (lt (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_P_1MiB)))
                      (const_int 4)
                      (const_int 8)))
    (set (attr "far_branch")
-       (if_then_else (and (ge (minus (match_dup 2) (pc)) (const_int -1048576))
-                          (lt (minus (match_dup 2) (pc)) (const_int 1048572)))
+       (if_then_else (and (ge (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_N_1MiB))
+                          (lt (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_P_1MiB)))
                      (const_int 0)
                      (const_int 1)))]
 )
@@ -829,13 +849,17 @@ (define_insn "aarch64_cbz<optab><mode>1"
   }
   [(set_attr "type" "branch")
    (set (attr "length")
-       (if_then_else (and (ge (minus (match_dup 1) (pc)) (const_int -1048576))
-                          (lt (minus (match_dup 1) (pc)) (const_int 1048572)))
+       (if_then_else (and (ge (minus (match_dup 1) (pc))
+                              (const_int BRANCH_LEN_N_1MiB))
+                          (lt (minus (match_dup 1) (pc))
+                              (const_int BRANCH_LEN_P_1MiB)))
                      (const_int 4)
                      (const_int 8)))
    (set (attr "far_branch")
-       (if_then_else (and (ge (minus (match_dup 2) (pc)) (const_int -1048576))
-                          (lt (minus (match_dup 2) (pc)) (const_int 1048572)))
+       (if_then_else (and (ge (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_N_1MiB))
+                          (lt (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_P_1MiB)))
                      (const_int 0)
                      (const_int 1)))]
 )
@@ -869,13 +893,17 @@ (define_insn "*aarch64_tbz<optab><mode>1"
   }
   [(set_attr "type" "branch")
    (set (attr "length")
-       (if_then_else (and (ge (minus (match_dup 1) (pc)) (const_int -32768))
-                          (lt (minus (match_dup 1) (pc)) (const_int 32764)))
+       (if_then_else (and (ge (minus (match_dup 1) (pc))
+                              (const_int BRANCH_LEN_N_32KiB))
+                          (lt (minus (match_dup 1) (pc))
+                              (const_int BRANCH_LEN_P_32KiB)))
                      (const_int 4)
                      (const_int 8)))
    (set (attr "far_branch")
-       (if_then_else (and (ge (minus (match_dup 1) (pc)) (const_int -1048576))
-                          (lt (minus (match_dup 1) (pc)) (const_int 1048572)))
+       (if_then_else (and (ge (minus (match_dup 1) (pc))
+                              (const_int BRANCH_LEN_N_1MiB))
+                          (lt (minus (match_dup 1) (pc))
+                              (const_int BRANCH_LEN_P_1MiB)))
                      (const_int 0)
                      (const_int 1)))]
 )
@@ -930,13 +958,17 @@ (define_insn "@aarch64_tbz<optab><ALLI:mode><GPI:mode>"
   }
   [(set_attr "type" "branch")
    (set (attr "length")
-       (if_then_else (and (ge (minus (match_dup 2) (pc)) (const_int -32768))
-                          (lt (minus (match_dup 2) (pc)) (const_int 32764)))
+       (if_then_else (and (ge (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_N_32KiB))
+                          (lt (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_P_32KiB)))
                      (const_int 4)
                      (const_int 8)))
    (set (attr "far_branch")
-       (if_then_else (and (ge (minus (match_dup 2) (pc)) (const_int -1048576))
-                          (lt (minus (match_dup 2) (pc)) (const_int 1048572)))
+       (if_then_else (and (ge (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_N_1MiB))
+                          (lt (minus (match_dup 2) (pc))
+                              (const_int BRANCH_LEN_P_1MiB)))
                      (const_int 0)
                      (const_int 1)))]
 
-- 
2.45.2

Reply via email to