Yes, of course, just this patch was prepared before last update for atomics. I'll update patch for sure.

Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team

16.02.2015 10:58, David Majnemer пишет:
================
Comment at: lib/CodeGen/CGStmtOpenMP.cpp:859
@@ +858,3 @@
+  else
+    CGF.EmitAtomicStore(ExprRValue, XLValue, /*isInit=*/false);
+  // OpenMP, 2.12.6, atomic Construct
----------------
Shouldn't you use `monotonic` ordering if the `omp write` isn't `seq_cst`?

http://reviews.llvm.org/D7369

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




_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to