[PATCH] D83708: Remove TwoAddressInstructinoPass::sink3AddrInstruction.

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 13 14:12:13 PDT 2020


nickdesaulniers added inline comments.


================
Comment at: llvm/test/CodeGen/X86/callbr-asm-sink.ll:24
+  %2 = getelementptr inbounds %struct1, %struct1* %0, i64 0, i32 1
+  callbr void asm sideeffect "# $0 $1 $2", "*m,X,~{memory},~{dirflag},~{fpsr},~{flags}"(i32* %2, i8* blockaddress(@klist_dec_and_del, %3))
+          to label %6 [label %3]
----------------
what is `$2` in this case if there's only 2 parameters explicitly passed into the `asm`? Maybe you can drop that here and in the `CHECK-NEXT` above?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83708





More information about the llvm-commits mailing list