arphaman added a comment.

Thanks! It looks pretty good, just minor nit. could you also update the 
language extension doc and release notes?



================
Comment at: clang/lib/Sema/SemaAttr.cpp:645
+
+  for (const PragmaAttributeEntry &Entry : PragmaAttributeStack.back().Entries)
+    if (!Entry.IsUsed) {
----------------
this for loop should have braces.


Repository:
  rC Clang

https://reviews.llvm.org/D53621



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

Reply via email to