[clang] [clang][Driver][HIP] Change OffloadingActionBuilder to respect the --no-gpu-bundle-output flag (PR #163834)

[alessandra simmons via cfe-commits]

================
@@ -0,0 +1,18 @@
+// RUN: %clang -ccc-print-phases -c -emit-llvm \
+// RUN:   --offload-arch=gfx900,gfx1030 -O3 -x hip %s \
+// RUN:   2>&1 | FileCheck %s --check-prefix=OFFLOAD
+
+// RUN: %clang -ccc-print-phases -c -emit-llvm \
+// RUN:   --gpu-bundle-output --offload-arch=gfx900,gfx1030 -O3 -x hip %s \
+// RUN:   2>&1 | FileCheck %s --check-prefix=OFFLOAD
+
+// RUN: %clang -ccc-print-phases -c -emit-llvm \
+// RUN:   --no-gpu-bundle-output --offload-arch=gfx900,gfx1030 -O3 -x hip %s \
+// RUN:   2>&1 | FileCheck %s --check-prefix=OFFLOAD2
+
+// OFFLOAD: clang-offload-bundler
+// OFFLOAD2-NOT: clang-offload-bundler
----------------
MixedMatched wrote:

> use a meaningful check line

ah! so sorry, I should have caught those

> also check the action entries for the NOBUNDLE case.

I tried looking through the -ccc-print-phases and the verbose outputs and I couldn't find a difference besides the order (which I thought might be flaky?) and the absence of the clang-offload-bundler command (which I'm already testing for). is there another way to test that which I'm missing?

https://github.com/llvm/llvm-project/pull/163834