jfb added a comment.

I've addressed Lang's comments and the re-audited all the new FIXME instances. 
This patch now actually drops errors on the floor instead of implicitly 
erroring out unless the error path is tested (which is what I had before). This 
hides bugs, but I left FIXMEs everywhere and it means the patch will be less 
disruptive because it's bug-compatible in the areas that aren't tested.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63518



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

Reply via email to