[PATCH] D56225: [HIP] Use nul instead of /dev/null when running on windows

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 10 07:07:21 PST 2019


yaxunl marked an inline comment as done.
yaxunl added inline comments.


================
Comment at: lib/Driver/ToolChains/HIP.cpp:210
   std::string BundlerTargetArg = "-targets=host-x86_64-unknown-linux";
-  std::string BundlerInputArg = "-inputs=/dev/null";
+  std::string BundlerInputArg = "-inputs=" NULL_FILE;
 
----------------
rjmccall wrote:
> How hard would it be to just implement the TODO and get away from a reliance on a null file?
I tried to make clang-offloader-bundler work without host target but it is not easy. Since it is shared with other languages e.g. OpenMP, it has a fundamental assumption that there is one host target. To make it work without host target may result in format changes that affecting HIP runtime, which is something I would like to avoid for now.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56225/new/

https://reviews.llvm.org/D56225





More information about the cfe-commits mailing list