[PATCH] D114590: [CodeGen] Emit alignment "Max Skip" operand for align directives

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 25 06:59:40 PST 2021


gchatelet requested changes to this revision.
gchatelet added a comment.
This revision now requires changes to proceed.

`Align` is a very generic class used in many contexts. Enriching it with `MaxBytesForAlignment` will break the mental model of everyone using it.
If you need the constraint for the `AsmPrinter` then pass it along manually.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114590



More information about the llvm-commits mailing list