skan added a comment. In D77728#1970922 <https://reviews.llvm.org/D77728#1970922>, @MaskRay wrote:
> In D77728#1970912 <https://reviews.llvm.org/D77728#1970912>, @skan wrote: > > > In D77728#1969788 <https://reviews.llvm.org/D77728#1969788>, @MaskRay wrote: > > > > > > Make -malign-branch-prefix-size an alias for -mpad-max-prefix-size (For > > > > GCC compatibility) > > > > > > I cannot find a discussion on -mpad-max-prefix-size. Is it a planned GNU > > > as option or GCC driver option? > > > > > > `-mpad-max-prefix-size` has nothing to do with GCC, it exposes the > > `-x86-pad-max-prefix-size` to clang > > > Then it is incorrect to say `HelpText<"Alias for -mpad-max-prefix-size">;// > Alias for GCC compatibility` Fine, we can have more discussion on this revision. D77628 <https://reviews.llvm.org/D77628> really has nothing to do with GCC/GAS compatibility, could you remove the "request change"? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77728/new/ https://reviews.llvm.org/D77728 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
