[PATCH] [OPENMP] Fix use of unsigned counters in loops with zero trip count.

Alexey Bataev a.bataev at hotmail.com
Wed Apr 22 04:26:03 PDT 2015


John, thanks for the review!


================
Comment at: lib/Sema/SemaOpenMP.cpp:2386
@@ +2385,3 @@
+Expr *OpenMPIterationSpaceChecker::BuildPreCond(Scope *S, Expr *Cond) const {
+  // Try to build LB ><>=<= UB.
+  bool Suppress = SemaRef.getDiagnostics().getSuppressAllDiagnostics();
----------------
rjmccall wrote:
> Cute, but more distracting than helpful. :)  Maybe "Try to build LB <op> UB, where <op> is <, >, <=, or >=."
John, thanks, I'll fix this. :)

http://reviews.llvm.org/D9103

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list