[PATCH] D108596: [NFC][AMDGPU] Reduce includes dependencies.

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 24 04:00:24 PDT 2021


dfukalov marked 2 inline comments as done.
dfukalov added a comment.

In D108596#2962039 <https://reviews.llvm.org/D108596#2962039>, @foad wrote:

> No objections from me, seems like a good cleanup. Do you plan to do more of this? What's the end goal?

Yes, I'm going to try additional NFC split out of parts like R600DAGToDAGISel, R600MCInstLower etc.

The end goal is to reduce AMDGPU and R600 source dependencies to (i) make partial rebuilds faster and (ii) prepare to move R600 target out at some point in future.

Perhaps we should think about additional split of some our huge modules in the sense of (i).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108596



More information about the llvm-commits mailing list