[PATCH] D17778: TypedError for recoverable error handling

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 15 15:18:07 PDT 2016


joker.eph added inline comments.

================
Comment at: unittests/Support/ErrorTest.cpp:406
@@ +405,3 @@
+                     "when an error exists!\"\\)")
+        << "Incorrect Expected error value";
+    consumeError(A.takeError());
----------------
How does it work with assertions disabled? (alternatively: where is this identified as "requires: assert"?)

To avoid the issue reported by David, you might roll your own "assert" so that you control the exact printed message?


http://reviews.llvm.org/D17778





More information about the llvm-commits mailing list