balazske added a comment.

The problem has not necessarily do something with macro expansion. If a 
function is imported in CTU mode and for any reason source locations inside it 
are compared against source locations in the original TU these locations are 
(from `SourceManager` point of view) in different TUs. But from TU point of 
view these are in the same TU because the AST import (AST import keeps the 
original filename and location in the new source location, so the new 
`SourceLocation` is in the same TU but different source file). Maybe this is a 
new case related to CTU mode in the compare function so it is correct to handle 
this case too (without assert).


Repository:
  rC Clang

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

https://reviews.llvm.org/D59934



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

Reply via email to