pzheng added a comment. In D87162#2257233 <https://reviews.llvm.org/D87162#2257233>, @MaskRay wrote:
> LGTM. > >> It's probably not used very common, but we do have some baremetal use cases >> still relying on these flags to be propagated to GCC. > > Thanks for the explanation! (Though I think in these cases calling `ld` or > `ld.lld` directly might be better?) > > And apologies about the friction but I hope the current behavior is better > than the previous whether many unrelated options can be forwarded and many > can cause warnings on GCC side. No worries. Hopefully, these flags are the only outliers. Thanks for reviewing the patch! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87162/new/ https://reviews.llvm.org/D87162 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
