[PATCH] D57896: Variable names rule

Mikhail Maltsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 19 03:43:17 PST 2019


miyuki added inline comments.


================
Comment at: llvm/docs/CodingStandards.rst:1065
   case 'J': {
-    if (Signed) {
-      Type = Context.getsigjmp_bufType();
-      if (Type.isNull()) {
-        Error = ASTContext::GE_Missing_sigjmp_buf;
+    if (signed) {
+      type = context.getsigjmp_bufType();
----------------
signed is a keyword, it can't be used as a variable name


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57896





More information about the llvm-commits mailing list