dcoughlin added a comment.

Do we have a radar for this? It sounds familiar.



================
Comment at: cfe/trunk/lib/StaticAnalyzer/Core/BugReporter.cpp:3313
 
+static bool isNoReturnBlock(const CFGBlock *Blk) {
+  if (Blk->hasNoReturnElement())
----------------
Maybe a better name for this would be "treatAsSink"? This way people won't use 
it as short-hand for whether the block has a no-return element.


Repository:
  rL LLVM

https://reviews.llvm.org/D35674



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

Reply via email to