[PATCH] D17741: adds __FILE_BASENAME__ builtin macro

Hal Finkel via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 3 07:20:30 PST 2016


hfinkel added a subscriber: hfinkel.
hfinkel added a comment.

In http://reviews.llvm.org/D17741#367113, @weimingz wrote:

> Add "-f__FILE__-prefix-to-remove" flag to support the trim of the prefix.
>  Passing special value __ALL_DIR__  to remove all dir parts.
>
> For example FILE is /a/b/c
>  -f__FILE__-prefix-to-remove=/a/ will cause FILE be expanded to b/c


I *really* don't like having `__FILE__` in the option name. That will look visually disjointing on the command line. How about: `-ffile-macro-prefix-to-remove`?


http://reviews.llvm.org/D17741





More information about the cfe-commits mailing list