lhames added a comment.

Heads up -- I think I just hit an error due to this while formatting a local 
commit:

  % ./clang/tools/clang-format/git-clang-format HEAD~1                          
                                                                         
  Assertion failed: (Line && Col && "Line and column should start from 1!"), 
function translateLineCol, file 
./llvm-project/clang/lib/Basic/SourceManager.cpp, line 1699.
  PLEASE submit a bug report to https://bugs.llvm.org/ and include the crash 
backtrace.            
  Stack dump:                                                                   
                                 
  0.      Program arguments: clang-format -lines=15:15 -lines=24:24 
-lines=46:47 -lines=73:74 -lines=82:82 -lines=117:117 -lines=128:128 
-lines=168:168 -lines=177:177 -lines=0:0 
llvm/lib/ExecutionEngine/JITLink/ELF_x86_64.cpp
  Stack dump without symbol names (ensure you have llvm-symbolizer in your PATH 
or set the environment var `LLVM_SYMBOLIZER_PATH` to point to it):              
                                                                 
  0  clang-format             0x0000000101390dad 
llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) + 61
  1  clang-format             0x000000010139135b 
PrintStackTraceSignalHandler(void*) + 27
  2  clang-format             0x000000010138f00b llvm::sys::RunSignalHandlers() 
+ 123                            
  3  clang-format             0x0000000101393fb8 SignalHandler(int) + 232       
                                 
  4  libsystem_platform.dylib 0x00007fff20639d7d _sigtramp + 29                 
                                 
  5  libsystem_platform.dylib 000000000000000000 _sigtramp + 
18446603339972764320
  6  libsystem_c.dylib        0x00007fff20549411 abort + 120                    
                                                                                
                                                                 
  7  libsystem_c.dylib        0x00007fff205487e8 err + 0
  8  clang-format             0x000000010146e898 
clang::SourceManager::translateLineCol(clang::FileID, unsigned int, unsigned 
int) const + 136
  9  clang-format             0x00000001011eb586 
clang::format::fillRanges(llvm::MemoryBuffer*, 
std::__1::vector<clang::tooling::Range, 
std::__1::allocator<clang::tooling::Range> >&) + 966                            
         
  10 clang-format             0x00000001011dd718 
clang::format::format(llvm::StringRef) + 1064                   
  11 clang-format             0x00000001011dcc1b main + 955                     
                                                                                
                                                                 
  12 libdyld.dylib            0x00007fff2060ff3d start + 1                      
                                 
  13 libdyld.dylib            0x000000000000000c start + 18446603339972935888   
                                                                                
                                                                 
  error: `clang-format -lines=15:15 -lines=24:24 -lines=46:47 -lines=73:74 
-lines=82:82 -lines=117:117 -lines=128:128 -lines=168:168 -lines=177:177 
-lines=0:0 llvm/lib/ExecutionEngine/JITLink/ELF_x86_64.cpp` failed

Reverting this patch fixed the error.

I am able to reliably reproduce this crash on Darwin by running:

  % git checkout fd26ca4e7515e7dd32ae02e777bd21693afc68ff
  % git am jitlink-table-manager-updates.patch
  % ./clang/tools/clang-format/git-clang-format HEAD~1

F19699034: jitlink-table-manager-updates.patch 
<https://reviews.llvm.org/F19699034>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111273

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to