[all-commits] [llvm/llvm-project] dbc76e: [NFC][llvm-exegesis] Benchmark: move DumpObjectToD...

[Roman Lebedev via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dbc76ef7915f925970f7bae1c5b79f3dcf55d85e
      https://github.com/llvm/llvm-project/commit/dbc76ef7915f925970f7bae1c5b79f3dcf55d85e
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2022-12-18 (Sun, 18 Dec 2022)

  Changed paths:
    M llvm/tools/llvm-exegesis/lib/BenchmarkRunner.cpp
    M llvm/tools/llvm-exegesis/lib/BenchmarkRunner.h
    M llvm/tools/llvm-exegesis/llvm-exegesis.cpp

  Log Message:
  -----------
  [NFC][llvm-exegesis] Benchmark: move DumpObjectToDisk handling into `runConfiguration()`

`getRunnableConfiguration()` may be executed in parallel,
and then this the output would become even less useful.


  Commit: 18da9a0cb35c3849e47625272e477dbc5544e25b
      https://github.com/llvm/llvm-project/commit/18da9a0cb35c3849e47625272e477dbc5544e25b
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2022-12-18 (Sun, 18 Dec 2022)

  Changed paths:
    M llvm/test/tools/llvm-exegesis/X86/latency/latency-CMOV32rr.s
    M llvm/tools/llvm-exegesis/llvm-exegesis.cpp

  Log Message:
  -----------
  [llvm-exegesis] Fix 'min' repetition mode in presence of missing measurements

This was a regression from 17e202424c021fd903950fec7a8b6cca2d83abce.
Previously we'd gracefully handle missing measurements,
but that handling got accidentally lost during the code move,
and we'd assert.

What we want to do, is to discard all measurements (from all repetitors
in a given config) if any of them failed, but do append the snippet,
and do emit the empty measurement.


Compare: https://github.com/llvm/llvm-project/compare/ba57d2ad4e9a...18da9a0cb35c