[PATCH] D95969: [WPD] Add an optional checking mode for debugging devirtualization
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 16 16:27:07 PST 2021
pcc added a comment.
In D95969#2567029 <https://reviews.llvm.org/D95969#2567029>, @tejohnson wrote:
> In D95969#2566916 <https://reviews.llvm.org/D95969#2566916>, @pcc wrote:
>
>> Can you use `SplitBlockAndInsertIfThen` instead of `versionCallSite` here? I personally don't find the name `versionCallSite` very intuitive here.
>
> There's a bunch of handling in versionCallSite for things like must tail calls and invoke instructions that I think I would end up needing to replicate, along with the general case. Calling versionCallSite makes things easy since it handles all of this. Would it help to expand the comment above the call?
You're just adding some code before the call, no? That means that you don't need to touch the call itself.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95969/new/
https://reviews.llvm.org/D95969
More information about the llvm-commits
mailing list