[PATCH] D129693: [ARM] Don't emit Arm speculation hardening thunks under Thumb and vice-versa

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 08:53:14 PDT 2022


dmgreen updated this revision to Diff 451167.
dmgreen added a comment.

The old version of this was only inserting one set of thunks even if the file contained multiple subtargets. This now uses InsertedThunks as a bitvector to distinguish between Thumb and Arm thunks being added. It should still only add each thunk once, but won't add both sets (arm/thumb) if they are not required.


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

https://reviews.llvm.org/D129693

Files:
  llvm/include/llvm/CodeGen/IndirectThunks.h
  llvm/lib/Target/AArch64/AArch64SLSHardening.cpp
  llvm/lib/Target/ARM/ARMSLSHardening.cpp
  llvm/lib/Target/X86/X86IndirectThunks.cpp
  llvm/test/CodeGen/ARM/speculation-hardening-sls-boththunks.ll
  llvm/test/CodeGen/ARM/speculation-hardening-sls.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D129693.451167.patch
Type: text/x-patch
Size: 8879 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220809/c556959e/attachment.bin>


More information about the llvm-commits mailing list