Mordante marked an inline comment as done. Mordante added a comment. Thanks for your feedback during the review.
================ Comment at: clang/include/clang/AST/Stmt.h:1180 + + /// \returns the likelihood of the branches of an if statement. + static Likelihood getLikelihood(const Stmt *Then, const Stmt *Else); ---------------- aaron.ballman wrote: > This only returns one value, so it can't return the likelihood of "the > branches". How about something along the lines of: > > `returns the likelihood of the 'then' branch of an 'if' statement. The 'else' > branch is required to determine whether both branches specify the same > likelihood, which impacts the result.` Good catch, I'll use your suggestion. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85091/new/ https://reviews.llvm.org/D85091 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
