[PATCH] D146376: Update static_assert message for redundant cases

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 22 02:20:34 PDT 2023


tbaeder added a comment.

Since we only handle `BinaryOperator`s in the toplevel assertion expression, I think it should just be safe to never diagnose for them if it's an `BO_LOr` opcode, so you should be able to just check for that in `DiagnoseStaticAssertDetails()`.


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