[clang] [clang-cl] Allow a colon after /Fo option (PR #87209)
Reid Kleckner via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 1 11:14:13 PDT 2024
rnk wrote:
That's interesting! I wonder how long the /F flags have accepted trailing colons. That form seems a lot more readable.
I went looking for examples of how to do this with fewer aliases, but it looks like this is what is done currently:
https://github.com/llvm/llvm-project/blob/main/clang/include/clang/Driver/Options.td#L5650
The only other prior art I could identify was the JoinedOrSeparate kind, which we could generalize to "JoinedOrEq" or "JoinedOrColon" in tablegen, and that would be the most maintainable, most general fix I can think of.
Regardless, I'm happy to approve this and other future patches to add the aliases in the meantime.
https://github.com/llvm/llvm-project/pull/87209
More information about the cfe-commits
mailing list