[PATCH] D139717: Revert "[Driver] Remove Joined -X"
Roy Sundahl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 13 06:20:38 PST 2022
rsundahl added a comment.
In D139717#3987963 <https://reviews.llvm.org/D139717#3987963>, @MaskRay wrote:
>> This change is breaking internal builds. We use the -Xfoo pattern but can now no longer manage whether we allow an unused -Xfoo option to pass as a warning or promote it to an error.
>
> Which `-Xfoo` is used? Do your internal builds reserve `-Xfoo` as a placeholder to do other non-upstream work? Then you can consider `-Gfoo` if your builds don't need the mips semantics.
We use -Xparser
> Is it possible to clean up the previously-ignored-with-a-warning `-Xfoo`?
Possibly. Maybe preferably. Builds are breaking now though. Dependent projects would have to be notified and changes scheduled. Was there a diff that we should have been part of? It looks like at least two of us are surprised with downstream consequences.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139717/new/
https://reviews.llvm.org/D139717
More information about the cfe-commits
mailing list