[PATCH] D101654: [HIP] Fix device lib selection

[Fangrui Song via Phabricator via cfe-commits]

MaskRay added inline comments.


================
Comment at: clang/test/Driver/hip-device-libs.hip:117
+// Test -fast-math
+// RUN: %clang -### -target x86_64-linux-gnu \
+// RUN:   --cuda-gpu-arch=gfx900 \
----------------
You can pack more arguments on one line.

Fewer line wraps actually make a test more readable IMO.


================
Comment at: clang/test/Driver/hip-device-libs.hip:165
+// FAST: "-mlink-builtin-bitcode" "{{.*}}oclc_daz_opt_off.bc"
+// FAST-SAME: "-mlink-builtin-bitcode" "{{.*}}oclc_unsafe_math_on.bc"
+// FAST-SAME: "-mlink-builtin-bitcode" "{{.*}}oclc_finite_only_on.bc"
----------------
You can use the `-SAME: {{^}} ` pattern in linux-cross.cpp


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

https://reviews.llvm.org/D101654