=?utf-8?q?Balázs_Kéri?= <balazs.k...@ericsson.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/124...@github.com>


Michael137 wrote:

> If this case is not handled, the following `setDescribedFunctionTemplate` 
> will run into an assertion. The case happens in the test code. What exactly 
> happens is that because of the `__get_first_arg` mismatch a field 
> `basic_string::_M_allocated_capacity` is missing from `basic_string` in the 
> To-AST. In this specific sequence of import calls the mismatch is found at 
> the template arguments when `getline` function template is imported. The test 
> was only a quick way to get a test (this reduced code was used to reproduce 
> the crash). With more work I can make a smaller test or even a test in 
> `ASTImporterTest`.

Ah thanks for the context. LGTM

Shorter test-case would be great, but not blocking this on it

https://github.com/llvm/llvm-project/pull/124273
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to