[PATCH] D148769: Split out `CodeGenTypes` from `CodeGen` for LLT/MVT

[Nico Weber via Phabricator via cfe-commits]

thakis added a comment.

In D148769#4311232 <https://reviews.llvm.org/D148769#4311232>, @dblaikie wrote:

> I wouldn't mind a follow-up commit that creates a new/separate directory (llvm/{lib,include/llvm}/CodeGenTypes) but that's probably a lot of unnecessary churn too, so I can appreciate the argument against it - if folks reckon it's better this way, that's OK too. It can just get a bit confusing when the directory/naming doesn't match the layering, etc. Might make it more likely someone regresses these boundaries by introducing new dependencies that break the layering here.

I'd really like to see this. That's how virtually all LLVM libraries are laid out, and it removes the PARTIAL_SOURCES_INTENDED kludge.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148769