EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

LGTM. For clang this patch should have no functionality change because clang 
allows attributes to appear only in the declaration.  GCC however was 
previously ignoring the attributes and only now will it mark those functions as 
hidden. I don't think this will be an issue though. Do you agree?


Repository:
  rL LLVM

http://reviews.llvm.org/D15395



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

Reply via email to