junior-jl wrote: Last commit changes: - Renamed `pattern_info` to `settings` in all of its uses. - Moved `HighlightSettings` declaration outside `for` loop. - Edited commit message as suggested.
> So you're asking about "stacked PRs" and I myself haven't tried this. LLVM > has recently enabled user branches on llvm/llvm-project so I think it is > possible using some of the tools out there, but I have not tried it. I would > look on the forums for anyone who's tried it, or ask on Discord for advice. ... Thanks for the detailed explanation. I'll look better into this. Question: Would be good/necessary to refactor `LookupSymbolInModule` into some smaller functions? At least separating the printing part? https://github.com/llvm/llvm-project/pull/76112 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
