aleksandr.urakov marked an inline comment as done.
aleksandr.urakov added inline comments.


================
Comment at: lit/SymbolFile/NativePDB/ast-types.cpp:77
+// FIXME: Should be located in the namespace `A`, in the struct `C<1>`.
+A::C<1>::D AC1D;
+
----------------
amccarth wrote:
> Is this a new problem because of your change?  Or is this an already existing 
> problem you discovered in the process?  Is the cause understood?  Should you 
> add a commented-out CHECK showing what the test would be when this problem is 
> fixed?
It's an existing problem, but I didn't go deep into it yet. I've added the test 
lines describing how it should be, thanks.


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

https://reviews.llvm.org/D60817



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to