This adds masked loads and stores back in, as an updated version of D66534. This time with an option to control whether they are generated, which is false until we can at least get them more efficient.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM, with a nit inline
llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp | ||
---|---|---|
494 ↗ | (On Diff #218733) | You pointed out to me that this is new, and more correct than before. ;-) |
502 ↗ | (On Diff #218733) | Better is to replace 128 with TTI->getRegisterBitWidth(true) |
Comment Actions
Thanks. Sorry, I was on a bit of a bug hunt, and this is the opposite of that ;) It's behind an option though, so should be fine.
llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp | ||
---|---|---|
494 ↗ | (On Diff #218733) | This will most likely need adjustment in the future, especially when we add narrowing/widening masked loads/stores. Hopefully OK for the time being. |