aaron.ballman added a comment.

In D136811#3913759 <https://reviews.llvm.org/D136811#3913759>, @jkorous wrote:

> @aaron.ballman We'd like to start making progress on the implementation in 
> parallel to iterating on the documentation and this is our first patch:
> https://reviews.llvm.org/D137346
>
> Since we'll have about 4 people working full-time on this it isn't reasonable 
> to expect you to review all our patches.
> My understanding is that you are interested mostly in the user model and the 
> general user interface.
> We can organize reviews for most patches among ourselves (with any extra 
> feedback more than welcome!) and would reach out to you specifically with the 
> things that are likely to interest you and/or where a broader consensus is 
> desirable.
> WDYT?

I think it's reasonable to start making progress on the implementation; we 
might debate some of the finer points, but if you're fine with the risk of 
needing to update the implementation later, I'd say "go for it!". My biggest 
request is: please try to keep this document in sync with reality so that we 
don't introduce confusion to the reviews.


Repository:
  rC Clang

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

https://reviews.llvm.org/D136811

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D136811: -Wunsafe... Aaron Ballman via Phabricator via cfe-commits

Reply via email to