steveire added a comment.

In D80499#2362028 <https://reviews.llvm.org/D80499#2362028>, @steveire wrote:

>> ! In D80499#2353187 <https://reviews.llvm.org/D80499#2353187>, @alexfh wrote:
>>  You should be ready for back and forth with this change, if users hit 
>> widespread issues not caught by tests. Maybe splitting it into separate 
>> pieces and involving check authors where practical may be reasonable.
>
> Ok, I'll do that when https://reviews.llvm.org/D80961 and 
> https://reviews.llvm.org/D82278 are merged.

@alexfh This is done in D91302 <https://reviews.llvm.org/D91302> and D91303 
<https://reviews.llvm.org/D91303>. Could you review?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80499

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

Reply via email to