[PATCH] D133194: rewording note note_constexpr_invalid_cast

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 2 05:49:28 PDT 2022


aaron.ballman requested changes to this revision.
aaron.ballman added a comment.
This revision now requires changes to proceed.

Thanks for the cleanup here! Precommit CI pointed out places where test coverage found issues, and you're missing new test coverage for the change. If you run the failing `constant-expression-cxx11.cpp` test in an asserts build, it'll help you to narrow down where the issue is. I think the problem is that there are more instances of issuing `diag::note_constexpr_invalid_cast` with the trailing argument `2` that weren't updated to pass the additional trailing argument you added.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133194



More information about the cfe-commits mailing list