[all-commits] [llvm/llvm-project] d8a353: [clang][deps] NFC: Remove else after early return

Jan Svoboda via All-commits all-commits at lists.llvm.org
Fri Nov 26 03:18:50 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d8a35387881bf408c19deeba7b5e3648dbbf467c
      https://github.com/llvm/llvm-project/commit/d8a35387881bf408c19deeba7b5e3648dbbf467c
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2021-11-26 (Fri, 26 Nov 2021)

  Changed paths:
    M clang/lib/Tooling/DependencyScanning/DependencyScanningFilesystem.cpp

  Log Message:
  -----------
  [clang][deps] NFC: Remove else after early return


  Commit: 12eafd944e0ffccae93402ddbe2855beb7a939ff
      https://github.com/llvm/llvm-project/commit/12eafd944e0ffccae93402ddbe2855beb7a939ff
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2021-11-26 (Fri, 26 Nov 2021)

  Changed paths:
    M clang/include/clang/Tooling/DependencyScanning/DependencyScanningFilesystem.h
    M clang/lib/Tooling/DependencyScanning/DependencyScanningFilesystem.cpp
    M clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp
    M clang/unittests/Tooling/DependencyScannerTest.cpp

  Log Message:
  -----------
  [clang][deps] NFC: Clean up wording (ignored vs minimized)

The filesystem used during dependency scanning does two things: it caches file entries and minimizes source file contents. We use the term "ignored file" in a couple of places, but it's not clear what exactly that means. This commit clears up the semantics, explicitly spelling out this relates to minimization.


Compare: https://github.com/llvm/llvm-project/compare/eafbaca97795...12eafd944e0f


More information about the All-commits mailing list