vabridgers added a comment.

I tested this change in the cases that previously failed for us, and this patch 
addresses our problems. It would be good to note in the commit header that this 
patch is intended to fix the problem first described by review D136886 
<https://reviews.llvm.org/D136886>. Finally, this should be submitted as part 
of a patch stack intended to address the 2 AST importer problems seen and 
corrected, originating from D133468 <https://reviews.llvm.org/D133468>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144273

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

Reply via email to