Hahnfeld added a comment.

Needs two small changes to work with current trunk


================
Comment at: lib/CodeGen/CGStmtOpenMP.cpp:2135-2136
@@ +2134,4 @@
+  const Expr *IfCond = nullptr;
+  if (auto C = S.getSingleClause(OMPC_if)) {
+    IfCond = cast<OMPIfClause>(C)->getCondition();
+  }
----------------
This now has to be `S.getSingleClause<OMPIfClause>()` and we can therefore omit 
the cast...

================
Comment at: lib/CodeGen/CGStmtOpenMP.cpp:2141-2142
@@ +2140,4 @@
+  const Expr *Device = nullptr;
+  if (auto C = S.getSingleClause(OMPC_device)) {
+    Device = cast<OMPDeviceClause>(C)->getDevice();
+  }
----------------
Likewise `S.getSingleClause<OMPDeviceClause>()` without the need for an extra 
cast


http://reviews.llvm.org/D11361



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to