[flang][Driver] Fix incorrect condition in test

[tarunprabhu]

The conditions in a test did not match the target that was being requested. This resulted in a test failure when building with -DTARGETS_TO_BUILD=X86. This is now fixed.

[llvmbot]

@llvm/pr-subscribers-flang-driver

Author: Tarun Prabhu (tarunprabhu)

Changes

The conditions in a test did not match the target that was being requested. This resulted in a test failure when building with -DTARGETS_TO_BUILD=X86. This is now fixed.

---

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

1 Files Affected:

• (modified) flang/test/Driver/print-supported-cpus.f90 (+2-2)

diff --git a/flang/test/Driver/print-supported-cpus.f90 b/flang/test/Driver/print-supported-cpus.f90
index 13688b193f9b92..60b725d4e1dcfc 100644
--- a/flang/test/Driver/print-supported-cpus.f90
+++ b/flang/test/Driver/print-supported-cpus.f90
@@ -18,11 +18,11 @@
 ! RUN:   %flang --target=aarch64-unknown-linux-gnu --print-supported-cpus 2>&1 \
 ! RUN:     | FileCheck %s --check-prefixes=AARCH64,CHECK \
 ! RUN: %}
-! RUN: %if x86-registered-target %{ \
+! RUN: %if aarch64-registered-target %{ \
 ! RUN:   %flang --target=aarch64-unknown-linux-gnu -mcpu=help 2>&1 \
 ! RUN:     | FileCheck %s --check-prefixes=AARCH64,CHECK \
 ! RUN: %}
-! RUN: %if x86-registered-target %{ \
+! RUN: %if aarch64-registered-target %{ \
 ! RUN:   %flang --target=aarch64-unknown-linux-gnu -mtune=help 2>&1 \
 ! RUN:     | FileCheck %s --check-prefixes=AARCH64,CHECK \
 ! RUN: %}

[tarunprabhu]

Apologies for not catching this before merging the original PR.

[banach-space]

Apologies for not catching this before merging the #117199.

You should blame the reviewers :)

LGTM, thanks!

[tblah]

Thanks!

[jplehr]

Is this likely going to fix the errors we are seeing in https://lab.llvm.org/staging/#/builders/105/builds/9796 ?

[tarunprabhu]

Is this likely going to fix the errors we are seeing in https://lab.llvm.org/staging/#/builders/105/builds/9796 ?

I haven't looked too closely at the failure, but the error suggests that this might.

[jplehr]

Thank you, it fixes the problem.