[PATCH] D39875: [PPC] Change i32 constant in store instruction to i64
Nemanja Ivanovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 10 03:27:12 PST 2017
nemanjai added a comment.
Thanks for taking care of this, I think it's quite useful and it should reduce the number of load immediate instructions and registers used. However, do we really need to expose a new API in `StoreSDNode`? Can we not just create a new truncating store using `SelectionDAG::getTruncStore()`? That would make this look a lot more like the rest of the DAG combines.
https://reviews.llvm.org/D39875
More information about the llvm-commits
mailing list