[PATCH] D114361: [MachineCSE] Add an option to enable global CSE

[wangpc via Phabricator via llvm-commits]

pcwang-thead marked 2 inline comments as done.
pcwang-thead added a comment.

In D114361#3148156 <https://reviews.llvm.org/D114361#3148156>, @dmgreen wrote:

> Thanks. What targets have you tested with this? And what kind of codesize differences have you observed?

RISCV. 
(And it seems that I need to modify other targets' tests)

The differences(in scope of functions):

- Some loads of immediates are redundanct.
- Some loads of global symbols are redundanct.
- etc.

These redundancies are in nonadjacent(non-local?) blocks , so they can't be eliminated according to `Heuristics #1` in `MachineCSE::isProfitableToCSE`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114361