[PATCH] D75300: Support prefix padding for alignment purposes (Relaxable instructions only)

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 13 21:34:18 PDT 2020


skan added a comment.

In D75300#1921757 <https://reviews.llvm.org/D75300#1921757>, @reames wrote:

> Per review comment, be more conservative about non-fully relaxed instructions.
>
> I think the previous code was in fact correct, but given it doesn't help any practical test I know of, being super conservative is reasonable for now.
>
> Can I get an LGTM on this?


I am running some tests for this patch, I will give it a LGTM once the tests are done.


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

https://reviews.llvm.org/D75300





More information about the llvm-commits mailing list