dsandersllvm wrote:

> `lldb/unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp` has some 
> unit-tests for `CPlusPlusNameParser` (and friends). If we added a constructor 
> to `CPlusPlusNameParser` which took a user-provided `LangOptions` (but 
> defaults to the existing one) that we could add a unit-test which constructs 
> the `CPlusPlusNameParser` with some `LangOptions` which triggers the issue 
> you were seeing. What do you think?

I think that's a good idea and I had a quick look at it today. It might make 
sense to be a separate PR but I'll see how big the change is first

https://github.com/llvm/llvm-project/pull/164284
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to