[Bug target/43961] [ARM thumb] branch out of range with thumb1_output_casesi

[mikpe at it dot uu.se]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43961



--- Comment #9 from Mikael Pettersson mikpe at it dot uu.se 2013-01-03 
08:54:35 UTC ---

(In reply to comment #8)

 Mikael, ping on this patch from June 2010 ... what happened in testing?



I've included this patch in my local 4.6-based branch since June 2010, and it's

tested w/o regressions on armv5te ever since (configured w/o

--disable-multilib, so I assume the test suite will also test Thumb-1 not just

ARM mode).



The patch apparently didn't work in a 4.5-based compiler, but I don't have any

notes explaining what the issue was.  4.5 is EOL anyway.



Somehow I forgot to forward-port it to 4.7 so I haven't tested it yet in my

4.7-based branch.  Will do that asap.

[Bug target/43961] [ARM thumb] branch out of range with thumb1_output_casesi

[jsm28 at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43961



--- Comment #8 from Joseph S. Myers jsm28 at gcc dot gnu.org 2013-01-03 
00:00:08 UTC ---

Mikael, ping on this patch from June 2010 ... what happened in testing?  And

shouldn't this bug be marked as a regression, given the identification of the

responsible revision?

[Bug target/43961] [ARM thumb] branch out of range with thumb1_output_casesi

[ramana at gcc dot gnu dot org]

--- Comment #7 from ramana at gcc dot gnu dot org  2010-07-03 00:21 ---
*** Bug 44603 has been marked as a duplicate of this bug. ***


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43961

[Bug target/43961] [ARM thumb] branch out of range with thumb1_output_casesi

[mikpe at it dot uu dot se]

--- Comment #5 from mikpe at it dot uu dot se  2010-06-22 12:22 ---
It's caused by r148770, which is when Richard Earnshaw added compressed switch
table support for Thumb-1.  See also:

http://gcc.gnu.org/ml/gcc-patches/2009-06/msg01698.html


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43961

[Bug target/43961] [ARM thumb] branch out of range with thumb1_output_casesi

[mikpe at it dot uu dot se]

--- Comment #6 from mikpe at it dot uu dot se  2010-06-22 12:28 ---
Created an attachment (id=20979)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20979action=view)
proposed fix for PR43961

Update ARM's ADDR_VEC_ALIGN to correctly describe that its
ASM_OUTPUT_CASE_LABEL will add a 2-byte alignment directive for Thumb-1
compressed switch tables.  This fixes the test cases (both the PR43961 and the
PR44603 one) for me.  Untested beyond that, will include in full bootstraps and
regtests shortly.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43961

[Bug target/43961] [ARM thumb] branch out of range with thumb1_output_casesi

[ramana at gcc dot gnu dot org]

--- Comment #4 from ramana at gcc dot gnu dot org  2010-05-10 16:03 ---
Confirmed - marking as a target bug .


-- 

ramana at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
  Component|c   |target
 Ever Confirmed|0   |1
 GCC target triplet|arm-elf |arm-elf,arm-eabi
  Known to fail||4.6.0
   Last reconfirmed|-00-00 00:00:00 |2010-05-10 16:03:26
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43961