[PATCH] D149522: [JITLink] Add target features to LinkGraph

[Job Noorman via Phabricator via llvm-commits]

jobnoorman added a comment.

In D149522#4318006 <https://reviews.llvm.org/D149522#4318006>, @lhames wrote:

> Yeah -- we definitely want to avoid the dependence on MC. One of the aims of ORC development over the next year or so is going to be splitting the library up so that ORC core depends only on libObject (and eventually, only on format-specific object file parsing libraries, but we don't have those yet. :) )

That makes sense. Unfortunately, it isn't as easy to avoid as I thought. The problem is that to get the features from an object file, we have to call `ObjectFile::getFeatures` which returns a `SubtargetFeatures` and we have to call one of its methods to convert it to a `std::vector<std::string>`. So I don't immediately see a way around the `MC` dependency.

On a related note: I wonder //why// `SubtargetFeatures` is part of `MC` and not of, say, `TargetParser` (like `Triple`).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149522