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

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineCSE.cpp:462
+  if (TII->isAsCheapAsAMove(*MI) && !EnableAggressiveMachineCSE &&
+      !TII->enableAggressiveMachineCSE(*MI->getMF())) {
     MachineBasicBlock *BB = MI->getParent();
----------------
Can this info be cached from runOnMachineFunction? No need to make a virtual call for something that wont' change per instruction.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114361