[all-commits] [llvm/llvm-project] fb42e5: [Driver] Split up huge arm-cortex-cpus.c test.

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


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

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

  Log Message:
  -----------
  [Driver] Split up huge arm-cortex-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/2832724/

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/D120876




More information about the All-commits mailing list