[PATCH] D15395: Add 3 more missing inline/visibility attributes

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 9 14:31:42 PST 2015


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





More information about the cfe-commits mailing list