[PATCH] D34072: Fix alignment bug in COFF emission.

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 12 15:30:04 PDT 2017


zturner added a comment.

So you're aligning the offset, not the absolute address.  In that it might be more intuitive to isntead of storing `Current`, which is actually a pointer, to store `Base` and `Offset` separately, and only operate on (and align) `Offset`, then compute `Base+Offset` every time you want to write?


https://reviews.llvm.org/D34072





More information about the llvm-commits mailing list