[all-commits] [llvm/llvm-project] 8f5bda: [Driver] Split up huge aarch64-cpus.c test.

Florian Hahn via All-commits all-commits at lists.llvm.org
Fri Mar 4 03:27:31 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8f5bdaf481c3f4e6876a5ae0e4d7c6ed2042e6a6
      https://github.com/llvm/llvm-project/commit/8f5bdaf481c3f4e6876a5ae0e4d7c6ed2042e6a6
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2022-03-04 (Fri, 04 Mar 2022)

  Changed paths:
    A clang/test/Driver/aarch64-cpus-1.c
    A clang/test/Driver/aarch64-cpus-2.c
    R clang/test/Driver/aarch64-cpus.c

  Log Message:
  -----------
  [Driver] Split up huge aarch64-cpus.c test.

This test file has grown to the point where it takes a huge amount of
time to run. At the moment, this test seems to consistently time out
when running in the pre-commit checks in Phabricator with a 10 minute
timeout. For example see
https://reviews.llvm.org/harbormaster/unit/view/2832723/

While splitting up the test file is not ideal, it is even more
undesirable to have huge test files that time out in common settings.

This patch splits up the test file roughly in the middle.

Reviewed By: dmgreen

Differential Revision: https://reviews.llvm.org/D120875




More information about the All-commits mailing list