[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