[all-commits] [llvm/llvm-project] 5933d2: [include-cleaner] Add a signal to down-rank export...

kadir ├žetinkaya via All-commits all-commits at lists.llvm.org
Wed Jul 5 06:41:34 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5933d265b72a8e9aade5edd68998a00dc4fbb359
      https://github.com/llvm/llvm-project/commit/5933d265b72a8e9aade5edd68998a00dc4fbb359
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2023-07-05 (Wed, 05 Jul 2023)

  Changed paths:
    M clang-tools-extra/include-cleaner/lib/FindHeaders.cpp
    M clang-tools-extra/include-cleaner/lib/TypesInternal.h
    M clang-tools-extra/include-cleaner/unittests/AnalysisTest.cpp
    M clang-tools-extra/include-cleaner/unittests/FindHeadersTest.cpp

  Log Message:
  -----------
  [include-cleaner] Add a signal to down-rank exporting headers

Currently exporter can have same relevance signals as the origin header
when name match signals don't trigger.
This patch introduces a tie braker signal to boost origin headers in
such cases, this is deliberately introduced with lower significance than
public-ness to make sure we still prefer a public-exporter instead of a
private-origin header.

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




More information about the All-commits mailing list