[PATCH] D42424: [InstCombine] Allow common type conversions to i8/i16
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 26 09:11:35 PST 2018
spatel added a comment.
In https://reviews.llvm.org/D42424#989058, @dmgreen wrote:
> Does anyone know if there a more sensible way to do this? Perhaps just for the ARM/AArch64 backends, not for all targets, no matter how odd they are. Or do we think this is the best way to do things?
The target-specific hack would be to change the data-layout for those targets, so i8 and i16 are legal ints. I don't know if that would cause any conflicts with legalization though. You could try that locally and see if anything breaks?
https://reviews.llvm.org/D42424
More information about the llvm-commits
mailing list