[PATCH] D38489: TargetMachine: Merge TargetMachine and LLVMTargetMachine

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 12 17:27:02 PDT 2017


MatzeB added a comment.

Turns out this doesn't work nice with the current layering. There are some tools that just link libTarget but not libCodeGen. Merging LLVMTargetMachine and TargetMachine however forces us to move TargetMachine into libCodeGen breaking such users.

I think when merging LLVMTargetMachine and TargetMachine we also have to be consequent and merge libTarget into libCodeGen to facilitate that. Not sure how people think about that, and I'm not sure I will find the time to prepare patches for all llvm users linking to libTarget.


Repository:
  rL LLVM

https://reviews.llvm.org/D38489





More information about the llvm-commits mailing list