teemperor accepted this revision.
teemperor added a comment.
Herald added a subscriber: JDevlieghere.

From what I can see the underlying problem with the import-std-module tests has 
been fixed at some point in the past (at least it doesn't reproduce anymore). I 
don't recall seeing a fix for the related issue, but maybe a libc++ change just 
moved things around so that we no longer try to call the wrong function. I will 
try to reland this and if the bots are happy then this seems good.

I wish I could say I finally got through my TODO list far enough to end up back 
here, but we actually do have a bug that is fixed by this patch. I'll add the 
respective tests as follow-up to the reland.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73191

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

Reply via email to