ygribov added inline comments. ================ Comment at: lib/StaticAnalyzer/Checkers/VforkChecker.cpp:149 @@ +148,3 @@ + + // see if it's an ordinary assignment + do { ---------------- ygribov wrote: > a.sidorin wrote: > > You can use early return to escape do{}. > In this particular case - yes. But what's wrong about original > do-while-false? I thought it's a common idiom... FYI there are several uses of do-while-false in CSA codebase.
Repository: rL LLVM http://reviews.llvm.org/D14014 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits