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

[Rafael Auler via Phabricator via llvm-commits]

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