[PATCH] D94972: [SLP]Need to shrink the load vector after reordering.

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 10:36:40 PST 2021


echristo added a comment.

Hard to tell because there aren't any comments around any of the code that changed in either patch. Can you fix that in a separate change and actually document what everything is doing? :)

For this patch can you add a comment that explains what we're doing here as well. It's unclear.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94972/new/

https://reviews.llvm.org/D94972



More information about the llvm-commits mailing list