[Openmp-commits] [PATCH] D112156: [OpenMP] Ensure broken assumptions print once, not thousands of times.

Joseph Huber via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Oct 20 10:43:09 PDT 2021


jhuber6 added inline comments.


================
Comment at: openmp/libomptarget/DeviceRTL/src/Debug.cpp:44
+    if (hash != atomic::exchange(&AssertionFlag, hash, __ATOMIC_SEQ_CST))
+      PRINTF("ASSERTION failed: %s at %s, line %d\n", exp, file, line);
     __builtin_trap();
----------------
Unrelated, but why don't we use the standard format from `assert.h` along with using `PRETTY_FUNCTION`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112156/new/

https://reviews.llvm.org/D112156



More information about the Openmp-commits mailing list