[clang] [llvm] Ast importer visitors (PR #138838)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 20 05:29:20 PDT 2025
ganenkokb-yandex wrote:
> Code looks almost acceptable. I could not verify if the tests are sufficient (I am not familiar with this new syntax). Can you test this on large projects that use these types of expressions?
It is my approach. All fixes aside ASTImporter are came from real cases.
I covered with tests all code paths for new decls and expressions for concepts, except
```
ImportExprRequirement
...
if (FromTypeRequirement.isSubstitutionFailure())
--- how to get here? - it is compile time error?
```
Newly added VisitSubstNonTypeTemplateParmPackExpr, VisitPseudoObjectExpr, VisitCXXParenListInitExpr are also without tests.
I've faced them on my project, and looks like it is not because of newly implemented concepts imports. But I'm not sure. Should I move 'em to separate pull request?
https://github.com/llvm/llvm-project/pull/138838
More information about the llvm-commits
mailing list