[PATCH] D71238: Align non-fused branches within 32-Byte boundary (basic case)

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 18:49:46 PST 2019


reames added a comment.

In D71238#1776443 <https://reviews.llvm.org/D71238#1776443>, @skan wrote:

> The patch doesn't handle with the hard code case either,  but as far as I can see that change was not mentioned in the description of this patch.


Unless I'm misreading the original patch, the notion of "hard code" is only relevant to the prefix padding scheme.  Admittedly, I might be misreading, it isn't clearly stated anywhere in the original patch exactly what "hard code" is or what purpose it serves.  .


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71238





More information about the llvm-commits mailing list