[PATCH] D47730: [SelectionDAG]Reduce masked data movement chains and memory access widths pt3

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 19 05:29:33 PDT 2018


samparker added a comment.

Hi Diogo,

I will look at this again once the two parent changes have been committed, as I'm not familiar with these functions. I'm confused as to why there isn't a ReduceStoreWidth to pair with ReduceLoadWidth. These LoadOpStore functions have a lot of overlap and related functionality that I suspect can be refactored nicely.

cheers,
sam


https://reviews.llvm.org/D47730





More information about the llvm-commits mailing list