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

[Kan Shengchen via Phabricator via cfe-commits]

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