[PATCH] D26713: [ppc] Legalize the load of MVT::v4i8 into VSX register
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 1 14:29:38 PST 2016
hfinkel added a comment.
This is certainly a novel use of setLoadExtAction for an illegal type ;) -- I worry that this is a bit hacky and not a technique we can use universally regardless. My inclination is that putting logic into PPCTargetTransformInfo would be better. X86TargetTransformInfo certainly has its fair share of cost tables, etc. to handle various situations the generic cost logic doesn't get quite right. I'm certainly open to hearing other opinions.
https://reviews.llvm.org/D26713
More information about the llvm-commits
mailing list