[all-commits] [llvm/llvm-project] e8e00e: [pseudo] benchmark cleanups. NFC

Sam McCall via All-commits all-commits at lists.llvm.org
Tue May 17 11:23:13 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e8e00e342c4fadc8355d1dfb8de86fb0a3dcd5f7
      https://github.com/llvm/llvm-project/commit/e8e00e342c4fadc8355d1dfb8de86fb0a3dcd5f7
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2022-05-17 (Tue, 17 May 2022)

  Changed paths:
    M clang-tools-extra/pseudo/benchmarks/Benchmark.cpp

  Log Message:
  -----------
  [pseudo] benchmark cleanups. NFC

- add missing benchmark for lex/preprocess steps
- name benchmarks after the function they're benchmarking, when appropriate
- remove unergonomic "run" prefixes from benchmark names
- give a useful error message if --grammar or --source are missing
- Use realistic example of how to run, run all benchmarks by default.
  (for someone who doesn't know the commands, this is the most useful action)
- Improve typos/wording in comment
- clean up unused vars
- avoid "parseable stream" name, which isn't a great name & not one I expected
  to escape from ClangPseudoMain

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




More information about the All-commits mailing list