From 1d6c3e29f6aa45914faa7be00a939b8f550f38e9 Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Mon, 12 Jun 2023 12:47:33 -0700 Subject: [PATCH] [Driver] Mark m_x86_Features_Group options as TargetSpecific so that they get an error on non-x86 targets. Follow-up to D151590. As a workaround for https://github.com/llvm/llvm-project/issues/63270, we don't report an error for -msse4.2. --- clang/include/clang/Driver/Options.td | 6 ++++++ clang/test/Driver/x86-target-features.c | 7 +++++++ 2 files changed, 13 insertions(+) diff --git a/clang/include/clang/Driver/Options.td b/clang/include/clang/Driver/Options.td index fd23321..980d17b 100644 --- a/clang/include/clang/Driver/Options.td +++ b/clang/include/clang/Driver/Options.td @@ -4774,6 +4774,7 @@ foreach i = {0-7} in } // let Flags = [TargetSpecific] // X86 feature flags +let Flags = [TargetSpecific] in { def mx87 : Flag<["-"], "mx87">, Group; def mno_x87 : Flag<["-"], "mno-x87">, Group; def m80387 : Flag<["-"], "m80387">, Alias; @@ -4807,7 +4808,11 @@ def mssse3 : Flag<["-"], "mssse3">, Group; def mno_ssse3 : Flag<["-"], "mno-ssse3">, Group; def msse4_1 : Flag<["-"], "msse4.1">, Group; def mno_sse4_1 : Flag<["-"], "mno-sse4.1">, Group; +} // let Flags = [TargetSpecific] +// TODO: Make -msse4.2 TargetSpecific after +// https://github.com/llvm/llvm-project/issues/63270 is fixed. def msse4_2 : Flag<["-"], "msse4.2">, Group; +let Flags = [TargetSpecific] in { def mno_sse4_2 : Flag<["-"], "mno-sse4.2">, Group; def msse4 : Flag<["-"], "msse4">, Alias; // -mno-sse4 turns off sse4.1 which has the effect of turning off everything @@ -4980,6 +4985,7 @@ def mretpoline_external_thunk : Flag<["-"], "mretpoline-external-thunk">, Group< def mno_retpoline_external_thunk : Flag<["-"], "mno-retpoline-external-thunk">, Group; def mvzeroupper : Flag<["-"], "mvzeroupper">, Group; def mno_vzeroupper : Flag<["-"], "mno-vzeroupper">, Group; +} // let Flags = [TargetSpecific] // These are legacy user-facing driver-level option spellings. They are always // aliases for options that are spelled using the more common Unix / GNU flag diff --git a/clang/test/Driver/x86-target-features.c b/clang/test/Driver/x86-target-features.c index 71bdd2a..f34b0f8 100644 --- a/clang/test/Driver/x86-target-features.c +++ b/clang/test/Driver/x86-target-features.c @@ -354,6 +354,13 @@ // CRC32: "-target-feature" "+crc32" // NO-CRC32: "-target-feature" "-crc32" +// RUN: %clang -### --target=aarch64 -mcrc32 -msse4.1 -msse4.2 -mno-sgx %s 2>&1 | FileCheck --check-prefix=NONX86 %s +// NONX86: error: unsupported option '-mcrc32' for target 'aarch64' +// NONX86-NEXT: error: unsupported option '-msse4.1' for target 'aarch64' +/// TODO: This warning is a workaround for https://github.com/llvm/llvm-project/issues/63270 +// NONX86-NEXT: warning: argument unused during compilation: '-msse4.2' [-Wunused-command-line-argument] +// NONX86-NEXT: error: unsupported option '-mno-sgx' for target 'aarch64' + // RUN: %clang --target=i386 -march=i386 -mharden-sls=return %s -### -o %t.o 2>&1 | FileCheck -check-prefixes=SLS-RET,NO-SLS %s // RUN: %clang --target=i386 -march=i386 -mharden-sls=indirect-jmp %s -### -o %t.o 2>&1 | FileCheck -check-prefixes=SLS-IJMP,NO-SLS %s // RUN: %clang --target=i386 -march=i386 -mharden-sls=none -mharden-sls=all %s -### -o %t.o 2>&1 | FileCheck -check-prefixes=SLS-IJMP,SLS-RET %s -- 2.7.4