ilya-biryukov added a comment.

In D59376#1454748 <https://reviews.llvm.org/D59376#1454748>, @ymandel wrote:

> I propose that I create a new library in Core with `getExtendedRange()` and 
> remove it from FixIt. The other  utility functions that I need will also go 
> there.  We can separately investigate moving the remaining pieces of FixIt 
> into Core.


Moving this to `Core` makes sense to me, the only concern that I have is that 
it's actually a small and focused library as it stands today and adding utility 
functions to it would seem to cause some chaos.
But it's fine as long as the maintainers of the library are ok with this. I'll 
ask around to find out who's actually responsible for the library and get back 
to you...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59376



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

Reply via email to