[PATCH] D50936: [DAGCombiner] Allow divide by constant optimization on opaque constants.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 17 22:01:26 PDT 2018


chandlerc added a comment.

Nice! I like it when the fix is simple.



================
Comment at: test/CodeGen/X86/divide-by-constant.ll:334
+
+define { i64, i32 } @PR38622(i64) {
+; X32-LABEL: PR38622:
----------------
If you add `nounwind` to this, the asm will get a lot less verbose while still showing the relevant bits.


Repository:
  rL LLVM

https://reviews.llvm.org/D50936





More information about the llvm-commits mailing list