[PATCH] D46386: Adding __atomic_fetch_min/max intrinsics to clang

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 3 11:43:59 PDT 2018


jfb added inline comments.


================
Comment at: lib/Sema/SemaChecking.cpp:3098
+  case AtomicExpr::AO__atomic_fetch_umax:
+    IsMinMax = true;
+    Form = Arithmetic;
----------------
Should `__sync_fetch_and_min` and others also set `IsMinMax`?


Repository:
  rC Clang

https://reviews.llvm.org/D46386





More information about the cfe-commits mailing list