[Lldb-commits] [PATCH] D76964: Fix an issue where the IgnoreName function was not allowing "Class" to be looked up inside a namespace or other class.

Raphael Isemann via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Apr 21 05:50:35 PDT 2021


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

I think this review got kind of stuck. I think the only missing thing is simplifying the `self.dbg.CreateTarget` call in the test, but otherwise this looks good to me. I'm just going to accept this as the bit of cleanup doesn't need another revision.

(Sorry for the delay, I kept clicking on this review and just saw unaddressed comments so I thought this wasn't done yet).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76964



More information about the lldb-commits mailing list