[llvm] r183238 - ARM: Fix crash in ARM backend inside of ARMConstantIslandPass

JF Bastien jfb at google.com
Wed Jun 5 07:08:32 PDT 2013


> It should have the older "MOV (imm)" instructions though, shouldn't
> it? They seem to support 8-bits in both ARM and Thumb.
>
> I'm not even that sure 8-bit loads *should* be supported. We're
> unlikely to allocate entries on an 8-bit granularity which means we
> can set the rest of the pool entry so that "ldr r0,[wherever]" would
> be just as effective and probably no more expensive than the ldrb.

Exactly my point. It seems odd and sub-optimal to me that you would ever
have 8-bit pool entry. Emitting it is probably a bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130605/64d811f6/attachment.html>


More information about the llvm-commits mailing list