[PATCH] D76811: [X86] Refactor X86IndirectThunks.cpp to Accomodate Mitigations other than Retpoline [2/3]
Scott Constable via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 31 07:44:50 PDT 2020
sconstab added a comment.
By the way, I had initially implemented this patch with a pure virtual base class and a retpoline thunk (and later LVI thunk) class that implements the interface. However, I could not for the life of me structure the classes in a manner that would allow the compiler to devirtualize. Using CRTP admittedly sacrifices some readability, but it does not prevent the compiler from inlining `RetpolineThunkInserter`'s methods.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76811/new/
https://reviews.llvm.org/D76811
More information about the llvm-commits
mailing list