[PATCH] D72227: Add options for clang to align branches within 32B boundary

Kan Shengchen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 9 23:01:38 PST 2020


skan marked an inline comment as done.
skan added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:2200
+          "Control how the assembler should align branches with NOP or segment "
+          "override prefix. If the boundary's size is not 0, it should be a "
+          "power of 2 and no less than 32. Branches will be aligned to prevent "
----------------
MaskRay wrote:
> "It should be 0 or a power of 2 no less than 16."
> 
> Do you want to preclude 16?
> 
> Long descriptions should be placed in `DocBrief` values.
No, I don't want to preclude 16.


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

https://reviews.llvm.org/D72227





More information about the cfe-commits mailing list