Prazek added a comment.

In https://reviews.llvm.org/D19201#760151, @aaron.ballman wrote:

> As an FYI, there is a related check being implemented in clang currently; we 
> probably should not duplicate this effort. See 
> https://reviews.llvm.org/D33333.


I think that clang is the right place for this feature, but I am not sure if 
the patch has all the features (like checking if something will be catched, or 
not showing warning for conditional noexcepts, which as we have seen
could be a problem for some projects. There also might be some other corner 
cases that we didn't think about. 
Assuming that this patch is ready to land, I would propose to send it to trunk 
and remove it when the clang's patch will make it to the trunk. I am not sure 
how much time it will take for other patch to be ready, but maybe we could 
gather some usefull bug reports in the meantime and also Stanislaw would have a 
contribution.


https://reviews.llvm.org/D19201



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

Reply via email to