[PATCH] rename _LIBCPP_TYPE_VIS_ONLY to _LIBCPP_TEMPLATE_VIS

John McCall rjmccall at apple.com
Tue Sep 2 19:46:23 PDT 2014


On Sep 2, 2014, at 2:01 PM, Reid Kleckner <rnk at google.com> wrote:
> ping

I like the idea of renaming the macro, but (1) you should preserve the type vs. function difference in the name and (2) you should rename _LIBCPP_FUNC_VIS_ONLY as well.

So, I would suggest _LIBCPP_TEMPLATE_TYPE_VIS and _LIBCPP_TEMPLATE_FUNC_VIS.

Also, please don’t strip the annotations completely.  If the correct axis is template vs. non-template — and I think that’s correct — then you should be adding _LIBCPP_TYPE_VIS to the types.

Otherwise seems fine to me.

John.



More information about the llvm-commits mailing list