[PATCH] D16025: [RS4GC] Replace some asserts by similar code using report_fatal_error().

Manuel Jacob via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 11 19:27:58 PST 2016


mjacob added a comment.

At some point in the future I want to be able to compile the language I'm working on using a stable released version of LLVM, which is usually compiled with assertions disabled. Currently invalid input IR can lead to undefined (or at least unwanted) behaviour if assertions are disabled.

Assertions shouldn't be used for input validation. Instead they should be used for catching programming errors. If you think I've rewritten an assertion that falls into the latter category, we can discuss that, though.


http://reviews.llvm.org/D16025





More information about the llvm-commits mailing list