[PATCH] D37912: [OpenMP] Bugfix: output file name drops the absolute path where full path is needed.

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 15 12:03:01 PDT 2017


tra added a comment.

In https://reviews.llvm.org/D37912#872318, @gtbercea wrote:

> In https://reviews.llvm.org/D37912#872294, @tra wrote:
>
> > Shouldn't this temp .cubin file go into the temporary directory, as opposed to the same directory as the input file?
>
>
> That is indeed the intention. The filename already contains the "/tmp/" I just make sure that doesn't get dropped.


Is that guaranteed to always be the case? If not, you're just hiding the problem.
IMO, it would be better to construct a temporary path for the file, if that's the intention. All you need is TC.getDriver().GetTemporaryPath(...).


Repository:
  rL LLVM

https://reviews.llvm.org/D37912





More information about the cfe-commits mailing list