[PATCH] D50636: [DAGCombiner] Add X / X -> 1 & X % X -> 0 folds.

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 28 00:24:28 PDT 2018


jonpa added a comment.

> it looks like changing the urem to this will trigger the original bug and avoid the optimization Simon is adding here.

Ah, nice :-)  I tried it, and just like you found that it triggers the original bug. I updated the test, patch is: F7066154: pr32372.ll.patch <https://reviews.llvm.org/F7066154>


Repository:
  rL LLVM

https://reviews.llvm.org/D50636





More information about the llvm-commits mailing list