kosarev added a comment.

In https://reviews.llvm.org/D37826#887820, @rjmccall wrote:

> I assume I should wait on reviewing this until all of these smaller TBAA 
> patches land?


These small patches are actually part of this diff. Generally, It depends on 
how you would like it: if you can review this re-based version as a whole, then 
that would save us some time.

Otherwise, you may want to proceed by smaller pieces, in which case 
https://reviews.llvm.org/D38503 is the next proposed piece to review.

Alternatively, if https://reviews.llvm.org/D38503 looks too complicated, please 
let me know and I will break it down into even smaller patches.

Or, if acceptable, I could commit a series of really small and simple diffs 
that all result in what is presented in https://reviews.llvm.org/D38126 and 
rely on post-commit reviews.

I'm fine with either way, provided the changes go to the mainline in time 
manner.

Thanks a lot for reviewing!


https://reviews.llvm.org/D37826



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to