[PATCH] D99275: Add the TableGen assert statement, step 2

Paul C. Anagnostopoulos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 24 13:25:23 PDT 2021


Paul-C-Anagnostopoulos added inline comments.


================
Comment at: llvm/include/llvm/TableGen/Error.h:51
 
+LLVM_ATTRIBUTE_NORETURN void CheckAssert(SMLoc Loc, Init *Condition,
+                                                    Init *Message);
----------------
lattner wrote:
> Why is this marked LLVM_ATTRIBUTE_NORETURN?  It looks like it does return.  I think this is copy/pasto?
It's more of a stupido. It didn't register with me what it indicated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99275



More information about the llvm-commits mailing list