[PATCH] D57829: [CUDA][HIP] Disable emitting llvm.linker.options in device compilation
Artem Belevich via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 4 10:47:29 PST 2019
tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.
LGTM in general. Nit about test name and test scope.
================
Comment at: clang/test/CodeGenCUDA/linker-options.cu:1-5
+// RUN: %clang_cc1 -emit-llvm -o - -fcuda-is-device -fms-extensions -x hip %s \
+// RUN: -fno-autolink -triple amdgcn-amd-amdhsa \
+// RUN: | FileCheck -check-prefix=DEV %s
+// RUN: %clang_cc1 -emit-llvm -o - -fms-extensions -x hip %s -triple \
+// RUN: x86_64-pc-windows-msvc | FileCheck -check-prefix=HOST %s
----------------
This appears to be specific to HIP on windows. If that's intended, then the file should be renamed to something like `hip-ms-linker-options.cu`. If the changes in functionality apply on other platforms, or with CUDA, the file name can remain as is, but it would be great to add some test runs for the other uses cases.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57829/new/
https://reviews.llvm.org/D57829
More information about the cfe-commits
mailing list