https://github.com/Xazax-hun approved this pull request.

There are some discussions open on this PR e.g., whether access paths need 
kinds. But I don't think those are blockers, this is exploratory work and it is 
expected that some design decisions might be revisited along the way when we 
learn something new. I am OK with the current status of this PR.

Something for future PRs: I think it would be nice to have some sort of 
coverage statistics. While initially you do not aim to cover all of C++, it 
would be good to know what portion of nodes are missing from the fact 
generation when we run this on some real world code. That can give us some 
progress indicators. 

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

Reply via email to