[all-commits] [llvm/llvm-project] 29b95f: [clang][deps] NFC: Refactor inferred modules test

Jan Svoboda via All-commits all-commits at lists.llvm.org
Wed Feb 8 16:58:26 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 29b95f05738fed5bf9777e285ccb084ada54b9b1
      https://github.com/llvm/llvm-project/commit/29b95f05738fed5bf9777e285ccb084ada54b9b1
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2023-02-08 (Wed, 08 Feb 2023)

  Changed paths:
    R clang/test/ClangScanDeps/Inputs/frameworks/Inferred.framework/Frameworks/Sub.framework/Headers/Sub.h
    R clang/test/ClangScanDeps/Inputs/frameworks/Inferred.framework/Headers/Inferred.h
    R clang/test/ClangScanDeps/Inputs/modules_inferred_cdb.json
    R clang/test/ClangScanDeps/modules-inferred-explicit-build.m
    M clang/test/ClangScanDeps/modules-inferred.m

  Log Message:
  -----------
  [clang][deps] NFC: Refactor inferred modules test

This patch squashes two tests with identical inputs into a single test, and adopts the `split-file` utility. This allows us to remove `sed` invocation with multiple commands, where "s|-E|-x objective-c -E|g" could've caused issues if previous replacements injected path containing "-E".

Reviewed By: benlangmuir

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




More information about the All-commits mailing list