aaron.ballman added a comment.

I'm not certain where you're planning to go with this change (or is this the 
only change you're trying to make in this area?), so it's a bit hard to 
evaluate this patch. Can you explain a bit more about what you're ultimately 
trying to accomplish?

It might help if I had a better idea of which APIs you thought were ones that 
would help users (because my only real concern with this change is that the 
public interface for this class is rather unpleasant).


Repository:
  rC Clang

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

https://reviews.llvm.org/D61835



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

Reply via email to