annita.zhang added a comment. In D70157#1793280 <https://reviews.llvm.org/D70157#1793280>, @reames wrote:
> 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. Thanks for landing it! And happy holiday to all! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70157/new/ https://reviews.llvm.org/D70157 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits