Skip to content

Commit

Permalink
[X86][Driver] Try to fix the test msse2avx.c on non-X86 target
Browse files Browse the repository at this point in the history 
The test was added in #96860
  • Loading branch information
@KanRobert
KanRobert committed Jul 16, 2024
1 parent f30c09e commit d82c75e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions clang/test/Driver/msse2avx.c
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// RUN: %clang -### -c -march=x86-64 -Xassembler -msse2avx %s 2>&1 | FileCheck %s
// RUN: %clang -### -c -march=x86-64 -x assembler -Xassembler -msse2avx %s 2>&1 | FileCheck %s
// RUN: %clang -### -c -target x86_64 -march=x86-64 -Xassembler -msse2avx %s 2>&1 | FileCheck %s

This comment has been minimized.

Sign in to view

Copy link
@MaskRay

MaskRay Jul 16, 2024

Member

-target has been deprecated for a long time. New tests should use --target=. I can refactor a few tests.
// RUN: %clang -### -c -target x86_64 -march=x86-64 -x assembler -Xassembler -msse2avx %s 2>&1 | FileCheck %s

// CHECK: "-msse2avx"

// RUN: not %clang -### -c --target=aarch64 -march=armv8a -msse2avx %s 2>&1 | FileCheck --check-prefix=ERR %s
// RUN: not %clang -### -c -target aarch64 -march=armv8a -msse2avx %s 2>&1 | FileCheck --check-prefix=ERR %s
// ERR: error: unsupported option '-msse2avx' for target 'aarch64'

0 comments on commit d82c75e

Please sign in to comment.