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