NoQ added a comment.

FTR, we already have a similar Static Analyzer check, eg.: 
https://github.com/llvm-mirror/clang/blob/release_80/test/Analysis/dispatch-once.m#L15

Your check is a bit more aggressive but i don't see why didn't we do it that 
way in the first place :)


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

https://reviews.llvm.org/D67567



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

Reply via email to