Re: [RFA/ARM] Fix PR54974: Thumb literal pools don't handle PC rounding

2012-11-29 Thread Matthew Gretton-Dann
On 24 November 2012 00:27, Ramana Radhakrishnan wrote: > On Wed, Nov 21, 2012 at 7:59 PM, Matthew Gretton-Dann > wrote: [snip] >> The fix is to decrease the pool_range of all insns by 2 when generating >> Thumb code. There is no need to change neg_pool_range values as rounding >> down here will

Re: [RFA/ARM] Fix PR54974: Thumb literal pools don't handle PC rounding

2012-11-23 Thread Ramana Radhakrishnan
On Wed, Nov 21, 2012 at 7:59 PM, Matthew Gretton-Dann wrote: > All, > > The attached patch fixes PR54974. > > In Thumb when calculating the PC value for a literal load the value used is > the current PC rounded down to the nearest multiple of 4. The ARM backend > currently does not take this into

[RFA/ARM] Fix PR54974: Thumb literal pools don't handle PC rounding

2012-11-21 Thread Matthew Gretton-Dann
All, The attached patch fixes PR54974. In Thumb when calculating the PC value for a literal load the value used is the current PC rounded down to the nearest multiple of 4. The ARM backend currently does not take this into account when calculating literal pool placement. The fix is to decrea