sammccall added a comment.

OOC, Is there something prompting doing this immediately?
I *did* originally plan to mechanically move this and we *can* but...

The API isn't even approximately the right shape (FileID should go away as 
we've baked main-file assumption in, Roots needs to be paired with macro refs, 
we need the includes and IWYU pragma stuff...). Then we have a function with a 
ton of parameters and there's probably some grouping that should go on.

So I'd be more inclined to let this sit for a couple of weeks and let the 
implementation catch up...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136782

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

Reply via email to