[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