[PATCH] D70157: Align branches within 32-Byte boundary

annita.zhang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 16 22:00:05 PST 2019


annita.zhang added a comment.

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

> Here are the minutes from our phone call a few minutes ago.


Thanks for coordinating the meeting and having a clear summary. It helps a lot to accelerate the patch review. I really appreciate it!

> Annita will refresh current patch with two key changes.  1) Drop prefix support and simplify and 2) drop clang driver support for now.  Desire is to minimize cycle time before next iteration so that feedback on approach can be given while reviewers are still around.

Yes, we are working on it right now. Hopefully we can submit a new patch today or tomorrow.

> Philip will prototype directive parsing support.  Annita and Yuo (??) to handle coordination on syntax.

I suppose it's Annita and Fangrui

> Side note to Annita: For you to remove "hard code", you'll have to have a placeholder for the enable/disable interface.  That should probably be split and rebased in my patch.

Let's do it in your directive patch.


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

https://reviews.llvm.org/D70157





More information about the cfe-commits mailing list