[arm] Fix insn type of *thumb1_tablejump
authorRichard Sandiford <richard.sandiford@arm.com>
Tue, 17 Sep 2019 15:02:15 +0000 (15:02 +0000)
committerRichard Sandiford <rsandifo@gcc.gnu.org>
Tue, 17 Sep 2019 15:02:15 +0000 (15:02 +0000)
*thumb1_tablejump had type "no_insn", which doesn't seems to correspond
to its documented use:

  an insn which does not represent an instruction in the final output,
  thus having no impact on scheduling.

Indirect jumps use the same instruction and have type "branch",
so the patch uses "branch" here too.

2019-09-17  Richard Sandiford  <richard.sandiford@arm.com>

gcc/
* config/arm/thumb1.md (*thumb1_tablejump): Change type from
"no_insn" to "branch".

From-SVN: r275806

gcc/ChangeLog
gcc/config/arm/thumb1.md

index b050369e53548d009713582ed8d9eb1206873a29..b0a11853583f05a8ef9644c3fefb064aa80ade4c 100644 (file)
@@ -1,3 +1,8 @@
+2019-09-17  Richard Sandiford  <richard.sandiford@arm.com>
+
+       * config/arm/thumb1.md (*thumb1_tablejump): Change type from
+       "no_insn" to "branch".
+
 2019-09-17  Richard Sandiford  <richard.sandiford@arm.com>
 
        * array-traits.h (array_traits<T[N]>::size): Remove parameter name.
index b142bfcb837befebc0298713d3454ac33941c956..5c70200bef36f010f0fbcc3694661c2d95adb570 100644 (file)
   "TARGET_THUMB1"
   "mov\\t%|pc, %0"
   [(set_attr "length" "2")
-   (set_attr "type" "no_insn")]
+   (set_attr "type" "branch")]
 )
 
 (define_insn_and_split "thumb_eh_return"