[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