clayborg accepted this revision.
clayborg added inline comments.
This revision is now accepted and ready to land.
================
Comment at: Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:2106
+// a vtable entry) from overloads (which require distinct entries).
+static bool isOverload(clang::CXXMethodDecl *m1, clang::CXXMethodDecl *m2) {
+ // FIXME: This should detect covariant return types, but currently doesn't.
----------------
I don't like things that can crash when asserts are off. I don't see why we
wouldn't just check this, If someone does pass in a method from on AST and
another from another AST, what will happen? Crash somewhere else? Why would we
risk crashing or misbehaving here when it is so easy to check and avoid. I'll
leave it at your discretion to do what you think is right though since I know
clang does this all over.
Repository:
rL LLVM
https://reviews.llvm.org/D41997
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
