[PATCH] D126137: [X86] Add support for `-mharden-sls=all`
    Phoebe Wang via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat May 21 21:03:57 PDT 2022
    
    
  
pengfei added inline comments.
================
Comment at: llvm/lib/Target/X86/X86AsmPrinter.cpp:341
+  MCInst TmpInst;
+  TmpInst.setOpcode(X86::INT3);
+  AsmPrinter::emitBasicBlockEnd(MBB);
----------------
craig.topper wrote:
> Why is TmpInst created unconditionally and not in the if?
Good catch!
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126137/new/
https://reviews.llvm.org/D126137
    
    
More information about the llvm-commits
mailing list