hazohelet updated this revision to Diff 550707.
hazohelet marked 3 inline comments as done.
hazohelet added a comment.
Address comments from Hubert
- Bring back type prefix
- NFC stylistic changes
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155610/new/
https://reviews.llvm.org/D155610
Files:
clang/docs/ReleaseNotes.rst
clang/lib/Sema/SemaDeclCXX.cpp
clang/test/Lexer/cxx1z-trigraphs.cpp
clang/test/SemaCXX/static-assert-cxx26.cpp
clang/test/SemaCXX/static-assert.cpp
Index: clang/test/SemaCXX/static-assert.cpp
===================================================================
--- clang/test/SemaCXX/static-assert.cpp
+++ clang/test/SemaCXX/static-assert.cpp
@@ -268,7 +268,31 @@
return 'c';
}
static_assert(getChar() == 'a', ""); // expected-error {{failed}} \
- // expected-note {{evaluates to ''c' == 'a''}}
+ // expected-note {{evaluates to ''c' (0x63, 99) == 'a' (0x61, 97)'}}
+ static_assert((char)9 == '\x61', ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to ''\t' (0x09, 9) == 'a' (0x61, 97)'}}
+ static_assert((char)10 == '\0', ""); // expected-error {{failed}} \
+ // expected-note {{n' (0x0A, 10) == '<U+0000>' (0x00, 0)'}}
+ // The note above is intended to match "evaluates to '\n' (0x0A, 10) == '<U+0000>' (0x00, 0)'", but if we write it as it is,
+ // the "\n" cannot be consumed by the diagnostic consumer.
+ static_assert((signed char)10 == (char)-123, ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to '10 == '<85>' (0x85, -123)'}}
+ static_assert((char)-4 == (unsigned char)-8, ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to ''<FC>' (0xFC, -4) == 248'}}
+ static_assert((char)-128 == (char)-123, ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to ''<80>' (0x80, -128) == '<85>' (0x85, -123)'}}
+ static_assert('\xA0' == (char)'\x20', ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to ''<A0>' (0xA0, -96) == ' ' (0x20, 32)'}}
+ static_assert((char16_t)L'ã' == L"C̵ÌÌ¯Ì ÌÍÍ
ť̺"[1], ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to 'u'ã' (0x3086, 12422) == L'̵' (0x335, 821)'}}
+ static_assert(L"ï¼¼ï¼"[1] == u'\xFFFD', ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to 'L'ï¼' (0xFF0F, 65295) == u'�' (0xFFFD, 65533)'}}
+ static_assert(L"â¾"[0] == U'ð', ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to 'L'â¾' (0x26BE, 9918) == U'ð' (0x1F30D, 127757)'}}
+ static_assert(U"\a"[0] == (wchar_t)9, ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to 'U'\a' (0x07, 7) == L'\t' (0x09, 9)'}}
+ static_assert(L"§"[0] == U'Ã', ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to 'L'§' (0xA7, 167) == U'Ã' (0xD6, 214)'}}
/// Bools are printed as bools.
constexpr bool invert(bool b) {
Index: clang/test/SemaCXX/static-assert-cxx26.cpp
===================================================================
--- clang/test/SemaCXX/static-assert-cxx26.cpp
+++ clang/test/SemaCXX/static-assert-cxx26.cpp
@@ -298,3 +298,12 @@
Bad<int> b; // expected-note {{in instantiation}}
}
+
+namespace EscapeInDiagnostic {
+static_assert('\u{9}' == (char)1, ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to ''\t' (0x09, 9) == '<U+0001>' (0x01, 1)'}}
+static_assert((char8_t)-128 == (char8_t)-123, ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to 'u8'<80>' (0x80, 128) == u8'<85>' (0x85, 133)'}}
+static_assert((char16_t)0xFEFF == (char16_t)0xDB93, ""); // expected-error {{failed}} \
+ // expected-note {{evaluates to 'u'' (0xFEFF, 65279) == u'\xDB93' (0xDB93, 56211)'}}
+}
Index: clang/test/Lexer/cxx1z-trigraphs.cpp
===================================================================
--- clang/test/Lexer/cxx1z-trigraphs.cpp
+++ clang/test/Lexer/cxx1z-trigraphs.cpp
@@ -21,7 +21,7 @@
#if !ENABLED_TRIGRAPHS
// expected-error@11 {{}} expected-warning@11 {{trigraph ignored}}
-// expected-error@13 {{failed}} expected-warning@13 {{trigraph ignored}} expected-note@13 {{evaluates to ''?' == '#''}}
+// expected-error@13 {{failed}} expected-warning@13 {{trigraph ignored}} expected-note@13 {{evaluates to ''?' (0x3F, 63) == '#' (0x23, 35)'}}
// expected-error@16 {{}}
// expected-error@20 {{}}
#else
Index: clang/lib/Sema/SemaDeclCXX.cpp
===================================================================
--- clang/lib/Sema/SemaDeclCXX.cpp
+++ clang/lib/Sema/SemaDeclCXX.cpp
@@ -49,6 +49,7 @@
#include "llvm/ADT/ScopeExit.h"
#include "llvm/ADT/SmallString.h"
#include "llvm/ADT/StringExtras.h"
+#include "llvm/Support/ConvertUTF.h"
#include "llvm/Support/SaveAndRestore.h"
#include <map>
#include <optional>
@@ -16866,10 +16867,75 @@
AssertMessageExpr, RParenLoc, false);
}
+static void WriteCharTypePrefix(BuiltinType::Kind BTK, llvm::raw_ostream &OS) {
+ switch (BTK) {
+ case BuiltinType::Char_S:
+ case BuiltinType::Char_U:
+ break;
+ case BuiltinType::Char8:
+ OS << "u8";
+ break;
+ case BuiltinType::Char16:
+ OS << 'u';
+ break;
+ case BuiltinType::Char32:
+ OS << 'U';
+ break;
+ case BuiltinType::WChar_S:
+ case BuiltinType::WChar_U:
+ OS << 'L';
+ break;
+ default:
+ llvm_unreachable("Non-character type");
+ }
+}
+
+/// Convert character's value, interpreted as a code unit, to a string.
+/// The value needs to be zero-extended to 32-bits.
+/// FIXME: This assumes Unicode literal encodings
+static void WriteCharValueForDiagnostic(uint32_t Value, const BuiltinType *BTy,
+ unsigned TyWidth,
+ llvm::raw_ostream &OS) {
+ char Arr[UNI_MAX_UTF8_BYTES_PER_CODE_POINT];
+ char *Ptr = Arr;
+ BuiltinType::Kind K = BTy->getKind();
+
+ // This should catch Char_S, Char_U, Char8, and use of escaped characters in
+ // other types.
+ if (K == BuiltinType::Char_S || K == BuiltinType::Char_U ||
+ K == BuiltinType::Char8 || Value <= 0x7F) {
+ StringRef Escaped = escapeCStyle<EscapeChar::Single>(Value);
+ if (!Escaped.empty())
+ OS << Escaped;
+ else
+ OS << static_cast<char>(Value);
+ return;
+ }
+
+ switch (K) {
+ case BuiltinType::Char16:
+ case BuiltinType::Char32:
+ case BuiltinType::WChar_S:
+ case BuiltinType::WChar_U: {
+ if (llvm::ConvertCodePointToUTF8(Value, Ptr)) {
+ OS << StringRef(Arr, Ptr - Arr);
+ } else {
+ OS << "\\x"
+ << llvm::format_hex_no_prefix(Value, TyWidth / 4,
+ /*Upper=*/true);
+ }
+ break;
+ }
+ default:
+ llvm_unreachable("Non-character type is passed");
+ }
+}
+
/// Convert \V to a string we can present to the user in a diagnostic
/// \T is the type of the expression that has been evaluated into \V
static bool ConvertAPValueToString(const APValue &V, QualType T,
- SmallVectorImpl<char> &Str) {
+ SmallVectorImpl<char> &Str,
+ ASTContext &Context) {
if (!V.hasValue())
return false;
@@ -16884,13 +16950,38 @@
"Bool type, but value is not 0 or 1");
llvm::raw_svector_ostream OS(Str);
OS << (BoolValue ? "true" : "false");
- } else if (T->isCharType()) {
+ } else {
+ llvm::raw_svector_ostream OS(Str);
// Same is true for chars.
- Str.push_back('\'');
- Str.push_back(V.getInt().getExtValue());
- Str.push_back('\'');
- } else
+ // We want to print the character representation for textual types
+ const auto *BTy = T->getAs<BuiltinType>();
+ if (BTy) {
+ switch (BTy->getKind()) {
+ case BuiltinType::Char_S:
+ case BuiltinType::Char_U:
+ case BuiltinType::Char8:
+ case BuiltinType::Char16:
+ case BuiltinType::Char32:
+ case BuiltinType::WChar_S:
+ case BuiltinType::WChar_U: {
+ unsigned TyWidth = Context.getIntWidth(T);
+ assert(8 <= TyWidth && TyWidth <= 32 && "Unexpected integer width");
+ uint32_t CodeUnit = static_cast<uint32_t>(V.getInt().getZExtValue());
+ WriteCharTypePrefix(BTy->getKind(), OS);
+ OS << '\'';
+ WriteCharValueForDiagnostic(CodeUnit, BTy, TyWidth, OS);
+ OS << "' (0x"
+ << llvm::format_hex_no_prefix(CodeUnit, /*Width=*/2,
+ /*Upper=*/true)
+ << ", " << V.getInt() << ')';
+ return true;
+ }
+ default:
+ break;
+ }
+ }
V.getInt().toString(Str);
+ }
break;
@@ -16987,8 +17078,9 @@
Side->EvaluateAsRValue(DiagSide[I].Result, Context, true);
- DiagSide[I].Print = ConvertAPValueToString(
- DiagSide[I].Result.Val, Side->getType(), DiagSide[I].ValueString);
+ DiagSide[I].Print =
+ ConvertAPValueToString(DiagSide[I].Result.Val, Side->getType(),
+ DiagSide[I].ValueString, Context);
}
if (DiagSide[0].Print && DiagSide[1].Print) {
Diag(Op->getExprLoc(), diag::note_expr_evaluates_to)
Index: clang/docs/ReleaseNotes.rst
===================================================================
--- clang/docs/ReleaseNotes.rst
+++ clang/docs/ReleaseNotes.rst
@@ -132,6 +132,44 @@
of a base class is not called in the constructor of its derived class.
- Clang no longer emits ``-Wmissing-variable-declarations`` for variables declared
with the ``register`` storage class.
+- Clang now warns on unused variables declared and initialized in condition
+ expressions.
+ (`#61681: <https://github.com/llvm/llvm-project/issues/61681>`_)
+- When describing the failure of static assertion of `==` expression, clang prints the integer
+ representation of the value as well as its character representation when
+ the user-provided expression is of character type. If the character is
+ non-printable, clang now shows the escpaed character.
+ Clang also prints multi-byte characters if the user-provided expression
+ is of multi-byte character type.
+
+ *Example Code*:
+
+ .. code-block:: c++
+
+ static_assert("A\n"[1] == U'ð');
+
+ *BEFORE*:
+
+ .. code-block:: text
+
+ source:1:15: error: static assertion failed due to requirement '"A\n"[1] == U'\U0001f30d''
+ 1 | static_assert("A\n"[1] == U'ð');
+ | ^~~~~~~~~~~~~~~~~
+ source:1:24: note: expression evaluates to ''
+ ' == 127757'
+ 1 | static_assert("A\n"[1] == U'ð');
+ | ~~~~~~~~~^~~~~~~~
+
+ *AFTER*:
+
+ .. code-block:: text
+
+ source:1:15: error: static assertion failed due to requirement '"A\n"[1] == U'\U0001f30d''
+ 1 | static_assert("A\n"[1] == U'ð');
+ | ^~~~~~~~~~~~~~~~~
+ source:1:24: note: expression evaluates to ''\n' (0x0A, 10) == U'ð' (0x1F30D, 127757)'
+ 1 | static_assert("A\n"[1] == U'ð');
+ | ~~~~~~~~~^~~~~~~~
Bug Fixes in This Version
-------------------------
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
