[PATCH] D81967: [clang-tools-extra] Prevent linking to duplicate .a libs and dylib

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 16 14:19:15 PDT 2020


mgorny added a comment.
Herald added a subscriber: wuzish.

For the record, this seems to trigger some command-line option parser problem:

  ******************** TEST 'Clang Tools :: clang-tidy/infrastructure/invalid-command-line.cpp' FAILED ********************                          
  Script:                                                                                                                                            
  --                                                                                                                                                 
  : 'RUN: at line 1';   not clang-tidy --invalid-arg 2>&1 | FileCheck /home/mgorny/git/llvm-project/clang-tools-extra/test/clang-tidy/infrastructure$
  invalid-command-line.cpp                                                                                                                           
  --                                                                                                                                                 
  Exit Code: 1                                                                                                                                       
                                                                                                                                                     
  Command Output (stderr):                                                                                                                           
  --                                                                                                                                                 
  /home/mgorny/git/llvm-project/clang-tools-extra/test/clang-tidy/infrastructure/invalid-command-line.cpp:4:16: error: CHECK-NEXT: expected string no
  t found in input                                                                                                                                   
  // CHECK-NEXT: clang-tidy{{(\.exe)?}}: Did you mean '--extra-arg'?                                                                                 
                 ^                                                                                                                                   
  <stdin>:2:1: note: scanning from here                                                                                                              
  clang-tidy: Did you mean '--enable-pre'?                                                                                                           
  ^                                                                        
                                                                                                                                                     
  Input file: <stdin>                                                      
  Check file: /home/mgorny/git/llvm-project/clang-tools-extra/test/clang-tidy/infrastructure/invalid-command-line.cpp                                
                                                                                                                                                     
  -dump-input=help describes the format of the following dump.             
                                                                                                                                                     
  Full input was:                                                                                                                                    
  <<<<<<                                                                                                                                             
          1: error: [CommonOptionsParser]: clang-tidy: Unknown command line argument '--invalid-arg'. Try: 'clang-tidy --help'                       
          2: clang-tidy: Did you mean '--enable-pre'?                                                                                                
  next:4     X~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ error: no match found                                                                          
  >>>>>>                                                                                                                                             

I'm going to investigate further tomorrow.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81967/new/

https://reviews.llvm.org/D81967





More information about the cfe-commits mailing list