jhuber6 added inline comments.

================
Comment at: clang/test/Driver/cuda-bindings.cu:148
+// RUN: | FileCheck -check-prefix=D_ONLY %s
+// D_ONLY: "foo.o"
----------------
tra wrote:
> Ideally we want to test that the output file actually survives the 
> compilation. It's traky to do for in-tree tests as we don't have `ptxas`. 
> However, we should be able to add a script which would just touch whatever is 
> specified with `-o` argument and ignore the rest and then actually do run 
> clang with `--ptxas-path=/path/to/dummy-ptxas` and then verify file existence.
Yeah, I wasn't sure if there was a good way. I don't have any CUDA machines to 
test so I just went with the simplest version.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131278

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

Reply via email to