tejohnson added a comment.

I was reviewing the llvm/lib/* changes and note that the new checking is a mix 
of assert (debug or assert enabled compiler only) and fatal errors (both debug 
and release compilers). Is that intended? If these are not likely due to user 
input issues, then perhaps they should all be assert so that they are compiled 
out in release compilers?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91410

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

Reply via email to