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

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 27 09:44:32 PST 2020


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


================
Comment at: llvm/include/llvm/MC/MCFragment.h:279
+/// ValueSize fields are ignored; nops are always emitted.  For date, the
+/// Value is repeated ValueSize times.  Value is usually 0.
 class MCAlignFragment : public MCFragment {
----------------
skan wrote:
> "Value is repeated ValueSize times" , I am afraid this statement is not true. `ValueSize` is the size of the integer (in bytes) of `Value`, and the `Value` is repeated `FragmentSize / AF.getValueSize()` times.
You're correct.  I'm just going to remove this.  The comment change isn't really related to the patch.


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

https://reviews.llvm.org/D75203





More information about the llvm-commits mailing list