This is an archive of the discontinued LLVM Phabricator instance.

[interception] Switch to _type suffix instead of _f, NFC.
ClosedPublic

Authored by kubamracek on Oct 11 2018, 1:44 PM.

Details

Summary

Some Darwin functions have pairs like dispatch_apply and dispatch_apply_f so the added _f to interceptor types causes a clash. Let's add _type suffix instead.

Diff Detail

Repository
rL LLVM

Event Timeline

kubamracek created this revision.Oct 11 2018, 1:44 PM
Herald added a subscriber: Restricted Project. · View Herald TranscriptOct 11 2018, 1:44 PM

Why not something even longer / more obvious, e.g. _wrapped_real_func ?

delcypher accepted this revision.Oct 11 2018, 2:49 PM

LGTM. Although I do agree with @george.karpenkov 's point.

This revision is now accepted and ready to land.Oct 11 2018, 2:49 PM

I think it should still be relatively brief to avoid generating debug info bloat, if that makes sense? So personally I'm for _type.

@kubamracek do you need this committed or can you do it yourself or are you planning changes to it? It's been sitting in the review queue as accepted for a while, just trying to clean it up.

Okay, I'm going to land this as it is. I hope that _type is clear enough.

This revision was automatically updated to reflect the committed changes.