[PATCH] D47425: [GlobalISel][Legalizer] Fix i1s being sign extended instead of zero-extended

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 29 16:37:22 PDT 2018


aemerson added a subscriber: t.p.northover.
aemerson added a comment.

I'll re-do this patch to unconditionally zero-extend for returns and also fix up the G_STORE legalisation to always zero-extend too. Given we don't have any target hooks that give us the information we need, let's go with just matching SelectionDAG behaviour?

As for the zeroext attributes, from what I can tell we've had the distinction between Darwin and non-Darwin with this since the beginning of the AArch64 target, which was contributed by @t.p.northover. I'd rather not be messing with that unless I have to.


Repository:
  rL LLVM

https://reviews.llvm.org/D47425





More information about the llvm-commits mailing list