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

Philip Reames via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 20 12:34:55 PST 2019


reames added a comment.

I've gone ahead and landed the patch so that we can iterate in tree.  See commit 14fc20ca62821b5f85582bf76a467d412248c248 <https://reviews.llvm.org/rG14fc20ca62821b5f85582bf76a467d412248c248>.

I've also landed a couple of follow up patches to address issues which would have otherwise required iteration on the review.  See commits c148e2e2ef86f53391be459752511684424f331b <https://reviews.llvm.org/rGc148e2e2ef86f53391be459752511684424f331b>, 4024d49edc1598a6f8017df541147b38bf1e2818 <https://reviews.llvm.org/rG4024d49edc1598a6f8017df541147b38bf1e2818>, and 8b725f0459eee468ed7f9935fba3278fcb4997b1 <https://reviews.llvm.org/rG8b725f0459eee468ed7f9935fba3278fcb4997b1>.

I still see some room for further cleanup (i.e. the fragment range scheme and tests), but what's in is of reasonable quality.

There's a couple follow up patches which are probably called for, but I think we can work on these in parallel now.

1. We need to settle on assembler syntax.
2. We need a patch for the x86 MI to MC translation to mark regions unsafe to pad.  (Probably best to separate from the above for the moment.)
3. We can incrementally add support for prefix padding.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70157





More information about the cfe-commits mailing list