[PATCH] D27803: [ARM] GlobalISel: Load i1, i8 and i16 args from stack
Diana Picus via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 19 12:27:35 PST 2017
rovka updated this revision to Diff 85007.
rovka edited the summary of this revision.
rovka added a comment.
Herald added a subscriber: kristof.beyls.
I've been thinking about this a bit more and I think it's better to generate a G_LOAD, but for a size of 4 so that we preserve the extension performed by the caller.
This is achieved by changing the type of the virtual register that we're supposed to load into to a 32-bit scalar (which is sufficient for now, since we only support scalars at the moment). I've also fixed the size of the memory operand, which was wrong in the previous versions of the patch.
Let me know what you think.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 9154 bytes
Desc: not available
More information about the llvm-commits