[PATCH] [Thumb/Thumb2] Added restrictions on PC, LR, SP in the register list for PUSH/POP/LDM/STM

Tim Northover t.p.northover at gmail.com
Wed Nov 5 07:27:50 PST 2014


Hi Jyoti,

On 5 November 2014 04:41, Jyoti Allur <jyoti.allur at samsung.com> wrote:
> Will it be correct to define this function as  follows since 4-TZ gives the size of instructions in ITblock & CurPosition points to current instruction in IT block?
> bool lastInITBlock()
> {
>    unsigned TZ = countTrailingZeros(ITState.Mask);
>    return (ITState.CurPosition == 4 - TZ);
> }

I've not tested it, but I think that might be "3 - TZ". Up to that
boundary condition it looks sane though. Best way is just to try and
find out, I think!

Cheers.

Tim.



More information about the llvm-commits mailing list