[all-commits] [llvm/llvm-project] 30d8e0: [RemarkUtil] Refactor remarkutil tool to use a com...

Zain Jaffal via All-commits all-commits at lists.llvm.org
Fri Jul 28 01:21:57 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 30d8e0837d68a11430cc4c0f24cd743e14219a7d
      https://github.com/llvm/llvm-project/commit/30d8e0837d68a11430cc4c0f24cd743e14219a7d
  Author: Zain Jaffal <z_jaffal at apple.com>
  Date:   2023-07-28 (Fri, 28 Jul 2023)

  Changed paths:
    M llvm/tools/llvm-remarkutil/CMakeLists.txt
    A llvm/tools/llvm-remarkutil/RemarkConvert.cpp
    A llvm/tools/llvm-remarkutil/RemarkCount.cpp
    M llvm/tools/llvm-remarkutil/RemarkUtil.cpp
    A llvm/tools/llvm-remarkutil/RemarkUtilHelpers.cpp
    A llvm/tools/llvm-remarkutil/RemarkUtilHelpers.h
    A llvm/tools/llvm-remarkutil/RemarkUtilRegistry.cpp
    A llvm/tools/llvm-remarkutil/RemarkUtilRegistry.h

  Log Message:
  -----------
  [RemarkUtil] Refactor remarkutil tool to use a command registry.

In preperation to move all remark utilities into one tool. We use
command registry to breakdown each utility into a separate file.

For now we have 3 utilities for remarks
1. Convert: which is responsible for converting yaml remarks to
   bitstream and vice-versa
2. Count: Analyse remarks and report count. This currently only supports
   asm-remarks and annotation-summary remarks.
3. Diff remarks: Currently we only have a diff for size remarks using
   `llvm-remark-size-diff`

The first two utilites have been simplified and seperated into two
files. The following commit will move `llvm-remark-size-diff` and fold
it to be inside `llvm-remarkutil` as a subcommand

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




More information about the All-commits mailing list