[Lldb-commits] [PATCH] D64591: [Expression] Move IRDynamicChecks to ClangExpressionParser

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jul 11 16:21:19 PDT 2019

jingham accepted this revision.
jingham added a comment.
This revision is now accepted and ready to land.

That would be cleaner.

OTOH, the original reason for these checkers was to help people understand crashes in their expressions more clearly.  Supposedly, modern languages "don't have pointers" and can't have bad objects, so the kind of crashes this instrumentation was supposed to help with "can't happen" and checkers for such languages wouldn't be all that helpful...

So while cleaner, maybe generalizing this more fully isn't a high priority change?  In which case, just getting them out of generic code seems fine as a stopping point.  Your choice.



More information about the lldb-commits mailing list