[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