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

[JF Bastien]

> Presumably for the same reason why we use ldr for 'ptrtoint (i32* @a to
i32)' instead of movw/movt. Why do you ask? Do you think using ldrb is
incorrect?

I do think that it is incorrect unless the critical path is extremely
integer execution unit bound, and the memory side is completely free. I
think that it's better to always use immediate mov for 8-bit values.

Things are different for 16, 32 and 64 bit constants, it varies quite a bit
depending in which processor is measured and on which workload.

That being said 8-bit constant pool entries should be supported in some
fashion, not cause a crash. I just think it shouldn't get emitted :-)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130604/633d0210/attachment.html>