[PATCH] D26015: Correctly classify main file includes if there is a prefix added

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 11 03:56:56 PST 2017


alexfh added a comment.

Tests don't pass with this patch applied:

  $ ninja check-clang-tools                                                                              
  [22/23] Running the Clang extra tools' regression tests
  FAIL: Extra Tools Unit Tests :: clang-tidy/ClangTidyTests/IncludeInserterTest.DontInsertDuplicateIncludeEvenIfMiscategorized (427 of 489)
  ******************** TEST 'Extra Tools Unit Tests :: clang-tidy/ClangTidyTests/IncludeInserterTest.DontInsertDuplicateIncludeEvenIfMiscategorized' FAILED ********************
  Note: Google Test filter = IncludeInserterTest.DontInsertDuplicateIncludeEvenIfMiscategorized
  [==========] Running 1 test from 1 test case.
  [----------] Global test environment set-up.
  [----------] 1 test from IncludeInserterTest
  [ RUN      ] IncludeInserterTest.DontInsertDuplicateIncludeEvenIfMiscategorized
  1 warning and 1 error generated.
  LLVM ERROR: 'a/header.h' file not found
  foo, bar
  
  
  ********************
  Testing Time: 5.32s
  ********************
  Failing Tests (1):
      Extra Tools Unit Tests :: clang-tidy/ClangTidyTests/IncludeInserterTest.DontInsertDuplicateIncludeEvenIfMiscategorized
  
    Expected Passes    : 487
    Expected Failures  : 1
    Unexpected Failures: 1
  FAILED: tools/clang/tools/extra/test/CMakeFiles/check-clang-tools 
  cd /build/tools/clang/tools/extra/test && /usr/bin/python2.7 /src/utils/lit/lit.py -sv /build/tools/clang/tools/extra/test
  ninja: build stopped: subcommand failed.


https://reviews.llvm.org/D26015





More information about the cfe-commits mailing list