[PATCH] D94542: [X86] Default to -x86-pad-for-align=false to drop assembler difference with or w/o -g

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 18:26:30 PST 2021


skan added a comment.

In D94542#2494838 <https://reviews.llvm.org/D94542#2494838>, @MaskRay wrote:

> In D94542#2494774 <https://reviews.llvm.org/D94542#2494774>, @reames wrote:
>
>> Both the align and branch handling are "optimizations".  I object to one being enabled and the other disabled.  If you want them both on by default, fine.  If you want them both off by default, fine.  Having one off and one on is confusing.
>
> I can disable the other one, too. Due to `if (!X86PadForAlign && !X86PadForBranchAlign)`, disabling one suffices.

It looks good to me either to turn off both options by default or just turn off `x86-pad-for-align`, since `x86-pad-for-branch-align` doesn't work 
indeed if `x86-align-branch-boundary` or  `x86-align-branch` is disabled.  If you prefer to turn off both, remember to set  `x86-pad-for-branch-align` to true when both `x86-align-branch-boundary` and `x86-align-branch` have valid non-zero value.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94542/new/

https://reviews.llvm.org/D94542



More information about the llvm-commits mailing list