[PATCH] D17741: adds __FILE_BASENAME__ builtin macro

Ben Craig via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 29 14:57:28 PST 2016


bcraig added a subscriber: bcraig.
bcraig added a comment.

Note: this doesn't count as an official "LGTM".

The code change seems fine to me.  I think this has been implemented in gcc as well, but I don't recall for certain.  If this has been implemented in gcc, then I would expect the semantics to be the same.  If it hasn't been implemented in gcc, then we might want to pick a different name for the macro (e.g. __CLANG_FILE_BASENAME__).


http://reviews.llvm.org/D17741





More information about the cfe-commits mailing list