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

[Job Noorman via Phabricator via llvm-commits]

jobnoorman added a comment.

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

> In D149522#4318568 <https://reviews.llvm.org/D149522#4318568>, @jobnoorman wrote:
>
>> On a related note: I wonder //why// `SubtargetFeatures` is part of `MC` and not of, say, `TargetParser` (like `Triple`).
>
> Huh. That's a really good question. Maybe we can do this by moving `SubtargetFeatures`?

fyi: I posted <https://discourse.llvm.org/t/move-subtargetfeatures-out-of-mc/70401> a question about this on Discourse.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149522