NoQ added a comment.

Ok I added D136811 <https://reviews.llvm.org/D136811> as a parent revision but 
I don't think it makes sense to land D136811 <https://reviews.llvm.org/D136811> 
before this patch, given that D136811 <https://reviews.llvm.org/D136811>'s 
documentation does not *yet* reflect the actual state of things. I think we 
should start landing patches, and land the documentation later when we think it 
actually makes sense to inform people about the warning and encourage enabling 
it.


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

https://reviews.llvm.org/D137346

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

Reply via email to