[PATCH] D51083: [x86/SLH] Teach SLH to harden indirect branches and switches without retpolines.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 4 03:41:15 PDT 2018


chandlerc marked an inline comment as done.
chandlerc added a comment.

Thanks for the review, landing.



================
Comment at: llvm/test/CodeGen/X86/speculative-load-hardening-indirect.ll:353
+; X64-PIC-NEXT:  .LBB5_1: # %bb0
+; X64-PIC-NEXT:    leaq {{.*}}(%rip), %rsi
+; X64-PIC-NEXT:    cmpq %rsi, %rdx
----------------
rnk wrote:
> It would be nice if the test checked that it was using the correct label. I consider this {{.*}} scrub a bug in update_llc_test_checks.py. For every test I've added, I run the updater with --no_x86_scrip_rip or whatever the argument is.
Good suggestion, done.


Repository:
  rL LLVM

https://reviews.llvm.org/D51083





More information about the llvm-commits mailing list