ashi1 added a comment.
Thank you for this patch, it looks good, and I have a few minor
questions/comments.
================
Comment at: clang/test/Driver/hip-phases.hip:34
+// RDC-DAG: [[P6:[0-9]+]]: backend, {[[P5]]}, ir, (device-[[T]], [[ARCH]])
+// RDC-DAG: [[P10:[0-9]+]]: linker, {[[P6]]}, image, (device-hip, [[ARCH]])
----------------
should also use device-[[T]]? I think there are several locations here using
device-hip instead of device[[T]].
================
Comment at: clang/test/Driver/hip-phases.hip:253
+// L2-DAG: [[P0:[0-9]+]]: input, "{{.*}}obj1.o", object, (host-[[T:hip]])
+// RL2-DAG: [[P1:[0-9]+]]: clang-offload-unbundler, {[[P0]]}, object,
(host-[[T:hip]])
+// L2-DAG: [[P2:[0-9]+]]: input, "{{.*}}obj2.o", object, (host-[[T:hip]])
----------------
[[T]] is defined everytime here, should probably use [[T]] in subsequent checks?
================
Comment at: clang/test/Driver/hip-save-temps.hip:58
+
+// -fgpu-rdc link
+// RUN: touch %T/obj1.o
----------------
Should this link test be moved into a separate file? Also to cover without -o?
I may also be adding more tests to this case for static lib generation too.
================
Comment at: clang/test/Driver/hip-toolchain-rdc.hip:10
// RUN: --hip-device-lib-path=%S/Inputs/hip_multiple_inputs/lib2 \
-// RUN: -fuse-ld=lld -fgpu-rdc \
+// RUN: -fuse-ld=lld -fgpu-rdc -nogpuinc \
// RUN: %S/Inputs/hip_multiple_inputs/a.cu \
----------------
What is the difference between this file and hip-toolchain-rdc-separate.hip?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81861/new/
https://reviews.llvm.org/D81861
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
