[PATCH] D110143: [Utils] Replace llc with cat for tests

Sebastian Neubauer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 21 06:00:33 PDT 2021


sebastian-ne added a comment.

In D110143#3012137 <https://reviews.llvm.org/D110143#3012137>, @foad wrote:

>> Implement substitutions in common.py.
>
> Does that mean they can be removed from the individual update_* tools? I guess that can be done as follow-ups.

Most update scripts don’t apply these substitution so far. `update_cc_test_checks.py` doesn’t use the common functions to start the process, but it could use get- and applySubstitutions.
I’ll do a follow-up if this gets merged.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110143



More information about the llvm-commits mailing list