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

Job Noorman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 17 02:37:23 PDT 2023


jobnoorman added a comment.

Thanks for the reviews @lhames!

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

> Side note, it looks like https://reviews.llvm.org/D150549 got at least one thumbs up already. If/when that lands we would benefit from going back to using `SubtargetFeatures` in `LinkGraph`, since it'll save us copying the vector (no that this is going to cost much in practice).

Agreed. Once that one lands, I'll submit a `JITLink` patch for this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149522



More information about the llvm-commits mailing list