aaron.ballman added a comment.

> which might be problematic in some contexts.

I'd appreciate some more details on what those contexts are. It's a bit 
unpleasant to expose these APIs as part of the public interface because we 
typically *do* want deserialization. I'm not opposed to the changes, but we 
don't usually add APIs without any calls to the API. I assume you're planning 
to use these once they're in; are those uses from within Clang itself?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154328

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

Reply via email to