[PATCH] D70157: Align branches within 32-Byte boundary(NOP padding)

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 19 18:16:40 PST 2019


skan added a comment.

In D70157#1791347 <https://reviews.llvm.org/D70157#1791347>, @reames wrote:

> The general question is why a *range* of fragments can't be defined.  Computing the instruction size for the entire range then just requires walking from first to last fragment in the range summing the size of each.  If both instructions are within the same data fragment, then no relaxation is needed, and the size of both is simply the size of the data fragment.
>
> Unless I'm missing something here?


Thank you for your detailed explanation, the solution has been consistent with your suggestion now.


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

https://reviews.llvm.org/D70157





More information about the llvm-commits mailing list