[Lldb-commits] [PATCH] D70846: Expression eval lookup speedup by not returning methods in ManualDWARFIndex::GetFunctions

Levon Ter-Grigoryan via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 9 03:18:49 PST 2019

PatriosTheGreat added a comment.

It seems like with this CL find-basic-functions logic became inconsistent for macos target and pc-linux.
In cases of pc-linux it returns no function with --function-flags=full flag and in macos it returns 7 functions.

I think there are 3 ways to fix this:

1. Fix the find-basic-functions by splitting checks for mac and pc.
2. Make mac logic same as linux. Fix this on mac GetFunctions method as well.
3. Revert to old logic for eFunctionNameTypeFull and add one more enum value like "eFunctionNameTypeFullMangled" which will return only function_fullnames and will be supported only in pc version.

Which are the most consistent way for you?

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list