[PATCH] D64394: [MachineCSE][MachinePRE] Do not hoist common computations into hot BBs

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineCSE.cpp:875
+         "CandidateBB should dominate MBB1");
+  return MBFI->getBlockFreq(CandidateBB) <=
+         MBFI->getBlockFreq(MBB) + MBFI->getBlockFreq(MBB1);
----------------
Should this also say something like "if OptForMinSize then return true"? Under the assumption that pre will reduce the codesize.


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

https://reviews.llvm.org/D64394