[Driver][SYCL] Do not pass along code coverage and profiling to device (#5423)

Code Coverage and the enabling profiling options are not supported for
device compilations for SPIR-V.  Disable the passing of these options
to the device compilation only, but allow for these to continue to work
for the host side

Author: mdtoguchi

clang/lib/Driver/ToolChains/Clang.cpp

@@ -5960,7 +5960,8 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
   // NVPTX/AMDGCN doesn't support PGO or coverage. There's no runtime support
   // for sampling, overhead of call arc collection is way too high and there's
   // no way to collect the output.
-  if (!Triple.isNVPTX() && !Triple.isAMDGCN())
+  // Disable for SPIR-V compilations as well.
+  if (!Triple.isNVPTX() && !Triple.isAMDGCN() && !Triple.isSPIR())
     addPGOAndCoverageFlags(TC, C, D, Output, Args, SanitizeArgs, CmdArgs);
 
   Args.AddLastArg(CmdArgs, options::OPT_fclang_abi_compat_EQ);

clang/lib/Driver/ToolChains/SYCL.cpp

@@ -652,8 +652,6 @@ SYCLToolChain::TranslateArgs(const llvm::opt::DerivedArgList &Args,
       default:
         DAL->append(A);
         break;
-      case options::OPT_fcoverage_mapping:
-        break;
       }
     }
   }

clang/test/Driver/sycl-offload.cpp

@@ -30,10 +30,19 @@
 /// Check that -fcoverage-mapping is disabled for device
 // RUN: %clang -### -fsycl -fprofile-instr-generate -fcoverage-mapping -target x86_64-unknown-linux-gnu -c %s 2>&1 \
 // RUN:  | FileCheck -check-prefix=CHECK_COVERAGE_MAPPING %s
-// CHECK_COVERAGE_MAPPING: clang{{.*}} "-cc1" "-triple" "spir64-unknown-unknown"{{.*}} "-fsycl-is-device"{{.*}} "-fprofile-instrument=clang"
+// CHECK_COVERAGE_MAPPING: clang{{.*}} "-cc1" "-triple" "spir64-unknown-unknown"{{.*}} "-fsycl-is-device"{{.*}}
+// CHECK_COVERAGE_MAPPING-NOT: "-fprofile-instrument=clang"
 // CHECK_COVERAGE_MAPPING-NOT: "-fcoverage-mapping"
 // CHECK_COVERAGE_MAPPING: clang{{.*}} "-cc1" "-triple" "x86_64-unknown-linux-gnu"{{.*}} "-fsycl-is-host"{{.*}} "-fprofile-instrument=clang"{{.*}} "-fcoverage-mapping"{{.*}}
 
+/// Check that -fprofile-arcs -ftest-coverage is disabled for device
+// RUN: %clang -### -fsycl -fprofile-arcs -ftest-coverage -target x86_64-unknown-linux-gnu -c %s 2>&1 \
+// RUN:  | FileCheck -check-prefix=CHECK_TEST_COVERAGE %s
+// CHECK_TEST_COVERAGE: clang{{.*}} "-cc1" "-triple" "spir64-unknown-unknown"{{.*}} "-fsycl-is-device"{{.*}}
+// CHECK_TEST_COVERAGE-NOT: "-ftest-coverage"
+// CHECK_TEST_COVERAGE-NOT: "-fprofile-arcs"
+// CHECK_TEST_COVERAGE: clang{{.*}} "-cc1" "-triple" "x86_64-unknown-linux-gnu"{{.*}} "-fsycl-is-host"{{.*}} "-ftest-coverage" "-fprofile-arcs"
+
 /// check for PIC for device wrap compilation when using -shared or -fPIC
 // RUN: %clangxx -### -fsycl -target x86_64-unknown-linux-gnu -shared %s 2>&1 \
 // RUN:   | FileCheck -check-prefix=CHECK_SHARED %s