shafik added inline comments.
================ Comment at: clang/lib/AST/APValue.cpp:658 for (auto &Val : Inits) { int64_t Char64 = Val.getInt().getExtValue(); if (!isASCII(Char64)) ---------------- I believe to be fully correct we also need to add: ``` if (!Val.isInt()) return false; ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131466/new/ https://reviews.llvm.org/D131466 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits