[PATCH] D55565: [X86] Don't emit MULX by default with BMI2

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 12 05:56:18 PST 2018


RKSimon added a comment.

I've generalized https://bugs.llvm.org/show_bug.cgi?id=34232 to cover all possible cases where we should/could use MULX



================
Comment at: test/CodeGen/X86/stack-folding-bmi2.ll:31
   ;CHECK-LABEL: stack_fold_mulx_u64
-  ;CHECK:       mulxq {{-?[0-9]*}}(%rsp), %rax, %rcx {{.*#+}} 8-byte Folded Reload
+  ;CHECK:       mulq {{-?[0-9]*}}(%rsp) {{.*#+}} 8-byte Folded Reload
   %1 = tail call i64 asm sideeffect "nop", "=x,~{rax},~{rbx},~{rcx},~{rdx},~{rsi},~{rdi},~{rbp},~{r8},~{r9},~{r10},~{r11},~{r12},~{r13},~{r14},~{r15}"()
----------------
Fix this?


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

https://reviews.llvm.org/D55565





More information about the llvm-commits mailing list