On Mon, Aug 28, 2017 at 7:40 PM, Richard Smith - zygoloid via Phabricator
via cfe-commits <cfe-commits@lists.llvm.org> wrote:

> rsmith added inline comments.
>
>
> ================
> Comment at: test/SemaCXX/destructor.cpp:27
> +#define BE_THE_HEADER
> +#include __FILE__
> +
> ----------------
> Do we guarantee that `__FILE__` names a path that can be used to include
> the current file? In other tests, we add `-include %s` to the `RUN:` line
> to model this situation.
>

 `ack 'include __FILE__' test` finds plenty of hits at least, this is a
somewhat common pattern. (As far as I can remember, introduced by you :-)
-- I might be misremembering though.)


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

Reply via email to