[PATCH] D126116: [X86][BOLT][NFC] Use getOperandType to determine memory access size

Rafael Auler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 11:28:14 PDT 2022


rafauler added a comment.

No, this diff is not NFC. It's actually pretty impactful (and thanks for working on this Amir).

If you change that, shrink wrapping will recognize way more instructions and will analyze more functions, causing the optimization to activate more often.

Because of that, we need to update tests. We may want to coordinate landing this together with my patchset of 5 diffs, so I can update all tests only once, instead of multiple times, one per diff.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126116



More information about the llvm-commits mailing list