[PATCH] D70157: Align branches within 32-Byte boundary

Philip Reames via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 6 11:18:50 PST 2019


reames added a comment.

Recording something so I don't forget it when we get back to the prefix padding version.  The write up on the bundle align mode stuff mentions a concerning memory overhead for the feature.  Since the basic implementation techniques are similar, we need to make sure we assess the memory overhead of the prefix padding implementation.  See https://www.chromium.org/nativeclient/pnacl/aligned-bundling-support-in-llvm for context.  I don't believe this is likely to be an issue for the nop padding variant.


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

https://reviews.llvm.org/D70157





More information about the cfe-commits mailing list