[PATCH] D101817: [MC][X86] Automatic alignment for Macro-Op Fusion

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 7 22:54:56 PDT 2021


Amir marked 2 inline comments as done.
Amir added inline comments.


================
Comment at: llvm/include/llvm/MC/MCFragment.h:563-567
 /// Represents required padding such that a particular other set of fragments
 /// does not cross a particular power-of-two boundary. The other fragments must
 /// follow this one within the same section.
+/// If AvoidEndAlign is set, this fragment will emit a minimum size nop to
+/// prevent the fragment following it from ending at a given \p AlignBoundary.
----------------
skan wrote:
> The comments here is weired after you added the new usage to the fragment, you need to refine it.
The comments are up-to-date. Please take a closer look at AvoidEndAlign flag usage below


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101817



More information about the llvm-commits mailing list