[Driver][test] Clean up some AIX tests
authorFangrui Song <i@maskray.me>
Sat, 26 Feb 2022 01:06:24 +0000 (01:06 +0000)
committerFangrui Song <i@maskray.me>
Sat, 26 Feb 2022 01:06:24 +0000 (01:06 +0000)
commitcb1654ee4beedc875c25a95e7b98f1aaed0b9e35
tree6dc6fed6696dc340bbe2afbc55ad8a2288721b12
parentbc7aeea8542d48111f8cc923451b7c53347c75b5
[Driver][test] Clean up some AIX tests

* For `-###`, `-o %t.o` is unnecessary if we don't specifically test the
  output filename.
* --target= is the canonical spelling. -target is a legacy spelling which
  unfortunately cannot be removed because there are too many uses.
* -no-canonical-prefixes uses the dereferenced absolute path for the cc1
  command. For most tests "-cc1" is sufficient to identify the command line, no
  need to specifically test the "clang" command, and -no-canonical-prefixes can
  removed.
* --unwindlib= is the preferred spelling. -u is a short option taking a value,
  which means a -uwindlib= typo cannot be detected.

I recommend that you take a look at linux-cross.cpp. Testing include paths and
library paths in one RUN line is sometimes more readable than having separate
include/library tests.

Having separate RUN lines for misc features like -fdata-sections
(`aix-data-sections.c`) is wasteful. It may be better testing multiple
options in a single RUN command.

Reviewed By: jsji

Differential Revision: https://reviews.llvm.org/D119309
clang/test/Driver/aix-as.c
clang/test/Driver/aix-data-sections.c
clang/test/Driver/aix-err-options.c
clang/test/Driver/aix-ld.c
clang/test/Driver/aix-mcpu-default.c
clang/test/Driver/aix-object-mode.c
clang/test/Driver/aix-rtlib.c
clang/test/Driver/aix-toolchain-include.cpp