aganea added a comment. To ease reviewers comprehension you have to propose patches that diff against the master. People sometimes apply the patches locally before accepting them. This current patch is a subtraction after D69778 <https://reviews.llvm.org/D69778> is applied locally on master. It seems a bit complicated.
I think it is better if you closed this, and applied all the changes you mean to do on D69778 <https://reviews.llvm.org/D69778> and re-open. I think it is a good patch, it was already accepted, you only need to demonstrate that the edge cases where it was failing are solved. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74846/new/ https://reviews.llvm.org/D74846 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits