NoQ added a comment.

Also, similarly to `getenv()`, in these cases domain-specific knowledge may 
help suppress some unwanted reports. Eg., if a file has been opened 
successfully, this doesn't technically mean that it'll be open successfully 
again, but it makes it more likely, and the user does not necessarily care 
about time-of-check-time-of-use races. So maybe it'd make sense to eventually 
move some of that stuff to StreamChecker anyway. Maybe not, hard to tell, need 
to see the results.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135247

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

Reply via email to