tahonermann added a comment.

For what it's worth, Coverity has been running with this patch in place for at 
least a year and a half now.  I believe the only reason it wasn't committed was 
due to concerns with the test case in https://reviews.llvm.org/D9126.  I 
haven't had time to address those concerns (and don't expect to anytime soon).


Repository:
  rL LLVM

https://reviews.llvm.org/D9127



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

Reply via email to