steakhal wrote:

Hey, looks like a nice patch!

I only have one question. Have you considered overriding some other 
`getRuntimeDefinition` too? When I'm looking at the [CallEvent 
](https://clang.llvm.org/doxygen/classclang_1_1ento_1_1CallEvent.html) 
inheritance graph, there could be a couple other options too.
Don't get me wrong, the `SimpleFunctionCall::getRuntimeDefinition` looks like 
the right place. I'm just curious.

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

Reply via email to