search

Ericsson / clang

Cross Translation Unit analysis capability for Clang Static Analyzer. (Fork of official clang at http://llvm.org/git/clang)
http://clang.llvm.org/
Other
15 stars 10 forks source link

Import every Decl in lambda record. #577

Closed balazske closed 5 years ago

balazske commented 5 years ago

Previously only the fields were imported. Now every Decl is imported. This way the destructor decl is not missing after import. (Not sure if this makes improvement beneath SourceLocation equivalence. But now the imported lambda should be structural equivalent to the original that may be important.)

martong commented 5 years ago

Just a minor thing. Could we have a unittest for this? When we are going to upstream this fix they will require that anyway, so perhaps it is better to have right now.

gamesh411 commented 5 years ago

Ongoing review: https://reviews.llvm.org/D57740