[PATCH] D130894: [clang] Print more information about failed static assertions

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 9 00:56:15 PDT 2022


tbaeder added a comment.

I don't really want to bike-shed about the presentation for too long... I'm fine with just removing the parens, since we present it like that in the error message as well anyway:

  ./assert.cpp:6:1: error: static assertion failed due to requirement ''c' == 'a''
  static_assert('c' == 'a')



> Any tests with macros?

I can add some, but they should be handled transparently, with the usual "expanded from macro" note.


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

https://reviews.llvm.org/D130894



More information about the cfe-commits mailing list