[clang] Fix test to require x86

[tarunprabhu]

This test would fail if the x86 target had not been built. This fix makes the requirement on x86 explicit. Trailing whitespace has also been removed.

[llvmbot]

@llvm/pr-subscribers-clang

Author: Tarun Prabhu (tarunprabhu)

Changes

This test would fail if the x86 target had not been built. This fix makes the requirement on x86 explicit. Trailing whitespace has also been removed.

---

Full diff: https://github.com/llvm/llvm-project/pull/162318.diff

1 Files Affected:

• (modified) clang/test/OpenMP/nvptx_throw_trap.cpp (+2-2)

diff --git a/clang/test/OpenMP/nvptx_throw_trap.cpp b/clang/test/OpenMP/nvptx_throw_trap.cpp
index b13a09136c17d..ce1173831f7c0 100644
--- a/clang/test/OpenMP/nvptx_throw_trap.cpp
+++ b/clang/test/OpenMP/nvptx_throw_trap.cpp
@@ -1,4 +1,4 @@
-// REQUIRES: nvptx-registered-target
+// REQUIRES: nvptx-registered-target, x86-registered-target
 
 // RUN: %clang_cc1 -fopenmp -triple nvptx64 -fopenmp-is-target-device %s -S -Wno-openmp-target-exception -o - | FileCheck -check-prefix=DEVICE %s
 // RUN: %clang_cc1 -fopenmp -triple x86_64-pc-linux-gnu -fopenmp-is-target-device -fcxx-exceptions %s -S -Wno-openmp-target-exception -o - | FileCheck -check-prefix=HOST %s
@@ -8,6 +8,6 @@
 // HOST-NOT: trap;
 #pragma omp declare target
 void foo(void) {
-	throw 404; 
+	throw 404;
 }
 #pragma omp end declare target

[AntonRydahl]

Thanks for addressing this issue!