yxsamliu wrote:
> Being able to override a flag is a good thing to have, IMO. There are builds
> where the owner of the leaf targets do not have much control over which
> options are set by the "default" compilation, so they need to rely on being
> able to override preceding options.
>
> The fact that we didn't need it till now for this particular option, and that
> we may be able to work around this particular use case does not negate
> conceptual usefulness, but makes it a "nice to have" thing, rather than a
> "must fix that".
>
> While we're at that, perhaps we should consider leaving those legacy-style
> flags as is, and implement new-style overridable `--[no-]offload-{inc,lib}` ?
Yes I think this is an opportunity to make this change so that these two
options are more composable
https://github.com/llvm/llvm-project/pull/140106
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
