[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