[PATCH] [PATCH] OpenMP firstprivate clause

Alexey Bataev a.bataev at hotmail.com
Mon Sep 30 21:02:28 PDT 2013



================
Comment at: lib/Sema/SemaOpenMP.cpp:987
@@ +986,3 @@
+    //  class type.
+    while (Type.getNonReferenceType()->isArrayType()) {
+      Type = cast<ArrayType>(
----------------
Doug Gregor wrote:
> ASTContext::getBaseElementType() does this loop for you. Please use that.
Ok, fixed.

================
Comment at: lib/Sema/SemaOpenMP.cpp:1038
@@ +1037,3 @@
+      bool IsConstant = Type.isConstant(Context);
+      while (Type->isArrayType()) {
+        QualType ElemType =
----------------
Doug Gregor wrote:
> ASTContext::getBaseElementType() again.
Ok, fixed.


http://llvm-reviews.chandlerc.com/D1694



More information about the cfe-commits mailing list