aaron.ballman added inline comments.

================
Comment at: lib/Sema/SemaDecl.cpp:6609
@@ +6608,3 @@
+
+  if (NewVD->hasLocalStorage() && NewVD->hasAttr<InternalLinkageAttr>()) {
+    Diag(NewVD->getLocation(), diag::warn_internal_linkage_local_storage);
----------------
Is there a reason this change cannot go into SemaDeclAttr.cpp instead? That way 
we don't bother to attach the attribute in the first place.

================
Comment at: lib/Sema/SemaDeclAttr.cpp:3407
@@ +3406,3 @@
+                                             : ExpectedVariableOrFunction);
+    D->dropAttr<AlwaysInlineAttr>();
+  }
----------------
Why is this dropping AlwaysInlineAttr instead of returning a nullptr?


Repository:
  rL LLVM

http://reviews.llvm.org/D13925



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

Reply via email to