[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