[PATCH] D75203: [X86] Relax existing instructions to reduce the number of nops needed for alignment purposes

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 11 17:52:21 PST 2021


skan added a comment.

In D75203#2491618 <https://reviews.llvm.org/D75203#2491618>, @MaskRay wrote:

> I am thinking of whether we can use some properties of MCSymbol to filter out some MCSymbol's in `LabeledFragments` but cannot find one which works (for example, `IsUsedInReloc` is only used in ObjectWriter, which is after the assembler optimization).
>
> @condy: `-mllvm -x86-pad-for-align=false` is a workaround.

Option `-x86-pad-for-align` is for assembler optimization, I think we should set its default value to false and add some comments that it may interact with labels in text section.  If someone turn on it explictly, the difference of machine code will be fine since it's a assembler optimization.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75203



More information about the llvm-commits mailing list