bcardosolopes wrote:

Thanks for working on this issue! 

> It was mentioned in the issue one type per pr, but as this types suppose to 
> have same vTable class name, I think it should be ok to handle them together, 
> (let me know if not).

You can handle more cases but that also means you need to add more testcases as 
part of the PR. Because you are asking more questions about how to handle other 
cases, perhaps it might be better to do the most simple first and incrementally 
add more information

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

Reply via email to