ASDenysPetrov added a comment. @steakhal
> @ASDenysPetrov Do you still want to rework the API of the `assumeZero`? This patch more looks like NFC, being just refactored. Actually I see that if we find and fix the root cause, we can freely refuse this patch. Another thing I see is that this patch will work after a root cause fix as well. And the last one is that as long a root cause stays unfixed, clang will emit an assertion without this patch, at least for my testcase. So, I can't really evaluate this objectively. What do you think? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77062/new/ https://reviews.llvm.org/D77062 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits