[Lldb-commits] [PATCH] D113174: [lldb] Summary provider for char flexible array members

[Jim Ingham via Phabricator via lldb-commits]

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

Regex Type summary matching happens after all the ConstString format matching.  Enrico did it that way because the ConstString matching is so much quicker, so if we can find a match there we'll get to it more quickly...

So this patch moves the char * recognition from the beginning of the type summary matching to the end, and potentially makes it a slower match.

I doubt that this will be noticeable on modern systems, however, just something to keep in mind.

It also changes the order of search slightly.  I think this is observable: it would mean a regex that happens to match "char *" as well as other things used to not be chosen for "char *" because it would have hit the ConstString summary first.  Now it will match, because the built-in regex summary will be checked after the user added ones.

Again, I don't think this is a reason not to do the patch.  But something to keep in mind.  The code itself looks fine.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113174/new/

https://reviews.llvm.org/D113174