hans added inline comments.

================
Comment at: utils/TableGen/ClangAttrEmitter.cpp:338
@@ -310,1 +337,3 @@
+        OS << "\n\n  static const " << getType() << " Default" << 
getUpperName()
+           << " = " << Default << ";";
     }
----------------
Everything up to the " = " part is the same for the two branches of the if 
statements.

Maybe re-write this as

```
OS << "\n\n  static const .... = ";
if (getType() == "bool")
  OS << true or false..
else
  OS << Default..
```


Repository:
  rL LLVM

http://reviews.llvm.org/D20213



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to