This is an archive of the discontinued LLVM Phabricator instance.

Allow configuring InlinerThreshold in C bindings for the new pass manager
ClosedPublic

Authored by erthalion on May 31 2023, 12:50 PM.

Details

Summary

C bindings for the new pass manager seem to allow to set any option from LLVMPassBuilderOptions, except InlinerThreshold. Allow to configure it as well.

Diff Detail

Event Timeline

erthalion created this revision.May 31 2023, 12:50 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: hiraditya. · View Herald Transcript
erthalion requested review of this revision.May 31 2023, 12:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 31 2023, 12:50 PM
This revision is now accepted and ready to land.May 31 2023, 1:00 PM

Forgot to mention, that I don't have commit access.

is there an email/name you'd like me to use for the commit?

is there an email/name you'd like me to use for the commit?

The one mentioned in the account would do, Dmitrii Dolgov. Thanks!

it's standard practice to also have an email in commits. if you upload with arc it'll automatically use your local git commit email when I download the patch, otherwise you'll have to manually tell me (or send it to aeubanks <at> google <dot> com if you don't want to type it here)