[Lldb-commits] [PATCH] D137838: [RFC][Support] Move TargetParsers to new component

Nico Weber via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Nov 11 09:14:28 PST 2022


thakis added a comment.

This is a gigantic diff. I'd recommend keeping the .h files in the old place for v0 and make them just forwarding headers that include the header at the new location. That way, you don't have to update oodles of include lines in this patch, and it makes it a bit easier to see what's going on. (You can then update all the include lines in a trivial follow-up if this change goes through, and then remove the forwarding headers in Support, to cut the dependency you want to remove.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137838



More information about the lldb-commits mailing list