[PATCH] D87453: [llvm-readobj/elf][test] - Test all core note types properly.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 03:44:57 PDT 2020


grimar added a comment.

In D87453#2270790 <https://reviews.llvm.org/D87453#2270790>, @jhenderson wrote:

> If you haven't already, could you run this test on Windows and make sure it doesn't take an unreasonable amount of time (since there are a lot of process executions, which are expensive on Windows), please. In theory, the test could be written with a single large input testing each individual value. This would make the test check/input bigger, but would at least avoid 150 or so process executions within a single test.

It takes 1.63s in debug configuration for me. (i7-4790K at 4.2 Ghz, windows 8.1 64, 32 Gb Ram, SSD).

  -- Testing: 1 tests, 1 workers --
  PASS: LLVM :: tools/llvm-readobj/ELF/note-core.test (1 of 1)
  
  Testing Time: 1.63s
    Passed: 1

Is it OK or too much?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87453/new/

https://reviews.llvm.org/D87453



More information about the llvm-commits mailing list