[PATCH] D84250: [InstSimplify] ExpandBinOp should fold an expression only when it's safe

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 07:22:43 PDT 2020


aqjune added a comment.

Should we go to D84792 <https://reviews.llvm.org/D84792>'s direction?
If it is, I think we should mention that it is a workaround for the miscompilations including PR33165, and it can still be problematic in theory but chose to do it for performance.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84250



More information about the llvm-commits mailing list