[PATCH] D146376: Update static_assert message for redundant cases
Dávid Bolvanský via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 7 01:41:13 PDT 2023
xbolva00 added a comment.
So.. new diagnostic is:
<source>:4:1: error: static assertion failed due to requirement 'is_gitlab'
static_assert(is_gitlab and is_weekend);
^ ~~~~~~~~~
Okay, a dev changes is_gitlab to true, compiles it again and boom, new error: static assertion failed due to requirement 'is_weekend'. With previous version of this diagnostic one can see all problems immediatelly. So I am not sure if I consider this change as a real improvement.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146376/new/
https://reviews.llvm.org/D146376
More information about the cfe-commits
mailing list