Some minor cleanup comments, but otherwise this LGTM.
================
Comment at: lib/Sema/SemaOpenMP.cpp:987
@@ +986,3 @@
+ // class type.
+ while (Type.getNonReferenceType()->isArrayType()) {
+ Type = cast<ArrayType>(
----------------
ASTContext::getBaseElementType() does this loop for you. Please use that.
================
Comment at: lib/Sema/SemaOpenMP.cpp:1038
@@ +1037,3 @@
+ bool IsConstant = Type.isConstant(Context);
+ while (Type->isArrayType()) {
+ QualType ElemType =
----------------
ASTContext::getBaseElementType() again.
http://llvm-reviews.chandlerc.com/D1694
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
