================ @@ -66,7 +44,17 @@ class SerializationFormat static char ID; // For RTTIExtends. ---------------- jkorous-apple wrote:
> Yea, the argument was that in the llvm ecosystem, people expect dyn_cast and > friends to work in class hierarchies. I don't think it's true that any class hierarchy in clang implements RTTI. I prefer keeping things simple while we are getting off the ground. We are starting to work on clients of the framework in the coming months. That will give us a lot more clarity on what the framework should and shouldn't do. Until then, the best strategy, at least in my mind, is to keep things simple because removing/changing complex parts takes more work than possibly introducing them later. https://github.com/llvm/llvm-project/pull/180779 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
