[PATCH] D134671: [Driver] Prevent Mips specific code from claiming -mabi argument on other targets.
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 29 12:47:16 PDT 2022
MaskRay added a comment.
In D134671#3824672 <https://reviews.llvm.org/D134671#3824672>, @aaron.ballman wrote:
> In D134671#3824644 <https://reviews.llvm.org/D134671#3824644>, @nickdesaulniers wrote:
>
>> I don't think it's an issue for us to work around downstream, but this did regress support for `-mabi=ms` used in UEFI related build scripts.
>> https://github.com/ClangBuiltLinux/linux/issues/1725
>> Noting it in case others find their way here via bisection. Thanks to @nathanchance for the report.
>
> If that's intentional, should we call this out as a potentially breaking change in the release notes and post an announcement?
I feel that this is still a minor issue. If we consider this as potentially breaking change, then we'd add many many driver changes as potentially breaking.
I think we should wait for more evidence.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134671/new/
https://reviews.llvm.org/D134671
More information about the cfe-commits
mailing list