This is an archive of the discontinued LLVM Phabricator instance.

[Driver][DXC] Remove a bunch of options from DXC
ClosedPublic

Authored by bogner on Aug 15 2023, 4:33 PM.

Details

Summary

Remove DXCOption from a whole bunch of options that we probably won't
support in the DXC driver. The initial clang-dxc support just made
everything that was a "CoreOption" available, regardless of whether it
made sense. Here I don't remove all of them, but this makes a dent on
making the list a bit more sensible. We can easily add or remove more
if they make sense later.

Diff Detail

Event Timeline

bogner created this revision.Aug 15 2023, 4:33 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: mcrosier. · View Herald Transcript
bogner requested review of this revision.Aug 15 2023, 4:33 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 15 2023, 4:33 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
beanz accepted this revision.Aug 15 2023, 4:56 PM
This revision is now accepted and ready to land.Aug 15 2023, 4:56 PM
This revision was automatically updated to reflect the committed changes.