rjmccall added a comment.

It sounds like what you want is a diagnostic library that's almost completely 
abstracted over the kinds of entities that can be stored in a diagnostic, 
including the definition of a source location.  I don't think that's 
incompatible with this patch; there's no need to suggest reversion.

Source locations, supplemental source ranges, and fix-its are all still 
meaningful concepts in the driver, it's just that a source location is, say, an 
offset into a particular argument rather than a location in the file.  So the 
challenge for you is to come up with a way to abstract this that doesn't 
significantly make things worse for Clang.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84362

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

Reply via email to