miyuki added a comment.

Yes, I am still interested in working on this. For now, I've split out some 
changes that don't require any modifications in `SourceLocation` into 
https://reviews.llvm.org/D89705.

> A general comment is that there are a couple of functional changes in this 
> patch, where hash values are changing and data structures are being changed, 
> but they're buried in the noise of the refactoring that's going on at the 
> same time. I suggest splitting this up at least into two

Will do.


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

https://reviews.llvm.org/D69840

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D69840: [B... Duncan P. N. Exon Smith via Phabricator via cfe-commits
    • [PATCH] D6984... Mikhail Maltsev via Phabricator via cfe-commits

Reply via email to