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

Thu Jul 18 17:08:02 PDT 2019

anton-afanasyev added a comment.

Btw, this change breaks multiple (more than two) hoisting to common dominator. I've tested this patch for the original test case taken from here: https://bugs.llvm.org/show_bug.cgi?id=38917. There are several comparisons giving `96 > 40 + 10, 96 > 29 + 10, 96 > 18 + 10` (so no hoisting at all), meanwhile `96 < 97 = 40 + 29 + 18 + 10`.
However I do not see easy solution for this issue.

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

https://reviews.llvm.org/D64394