This is an archive of the discontinued LLVM Phabricator instance.

[clang] Make config-related options CoreOptions
ClosedPublic

Authored by mgorny on Sep 19 2022, 9:09 AM.

Details

Summary

Make --config, --no-default-config and --config-*-dir CoreOptions
to enable their availability to all clang driver modes. This improves
consistency given that the default set of configuration files is
processed independently of mode anyway.

Diff Detail

Event Timeline

mgorny created this revision.Sep 19 2022, 9:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 19 2022, 9:09 AM
Herald added a subscriber: StephenFan. · View Herald Transcript
mgorny requested review of this revision.Sep 19 2022, 9:09 AM
MaskRay accepted this revision.Sep 19 2022, 9:16 AM

Thanks!

This revision is now accepted and ready to land.Sep 19 2022, 9:16 AM

They don't need to be NoXarchOption if I am not mistaken.

They don't need to be NoXarchOption if I am not mistaken.

I don't know about that. The comment on top says "The option is a "driver"-only option, and should not be forwarded to other tools via -Xarch options.". Well, it's processed by driver only, isn't it?

This revision was landed with ongoing or failed builds.Sep 19 2022, 9:58 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 19 2022, 9:58 AM