Meinersbur wrote:

> IIRC it was unnecessary for the documentation because it gets built multiple 
> times, for the driver it's built once so it has to include all possible 
> variants the first time.
> 
> I have no preference for how it's done, if the `%Program` thing works then 
> fine. We only need a `ForVariants` if it's something people see when 
> interacting with Flang itself I think.

I didn't know this and experimented a bit. Indeed `%Program` in `HelpText` gets 
emitted as "Clang" in `flang-new --help`. `DocBrief` and `HelpText` are "Flang" 
for `FlangCommandLineReference.rst`.

So `DocBriefForVariant` would not add new functionality, but might be useful so 
`DocBrief[ForVariant]` and `HelpText[ForVariant]` can use the same syntax and 
avoid confusion. I will merge this patch and work on `DocBriefForVariant` after 
that.

https://github.com/llvm/llvm-project/pull/88932
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to