[PATCH] D122618: [Lint][Verifier] NFC: Rename 'Assert*' macros to 'Check*'.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 28 15:04:04 PDT 2022


nikic added a comment.

My reading on the "assert" terminology here was that we don't continue executing the rest of the function if a condition fails. Basically with the same implication as ASSERT in gunit tests (vs EXPECT, which reports an error but continues running the test).

But I don't particularly care either way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122618



More information about the llvm-commits mailing list