[PATCH] D120876: [Driver] Split up huge arm-cortex-cpus.c test.

Florian Hahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 3 01:17:45 PST 2022


fhahn created this revision.
fhahn added reviewers: simon_tatham, dmgreen, tmatheson, tyb0807.
Herald added a subscriber: kristof.beyls.
Herald added a project: All.
fhahn requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

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.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D120876

Files:
  clang/test/Driver/arm-cortex-cpus-1.c
  clang/test/Driver/arm-cortex-cpus-2.c
  clang/test/Driver/arm-cortex-cpus.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120876.412629.patch
Type: text/x-patch
Size: 77180 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220303/a5c8466c/attachment-0001.bin>


More information about the cfe-commits mailing list