[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