[PATCH] D45360: [MC][TableGen] Add optional libpfm counter names for ProcResUnits.

Andrea Di Biagio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 06:29:58 PDT 2018


andreadb added a comment.

In https://reviews.llvm.org/D45360#1059635, @RKSimon wrote:

> I'm a bit worried about creating a pfm dependency like this in the core - pfm doesn't come close to covering all targets that we'll want to check in the long term so it's likely that some targets will require a different library. (or raw msrs.....)
>
> Is there anyway that we can keep more of this in llvm-exegesis project - config files or embedding in source?


I tend to agree with Simon.
All of this info could live in a config file within exegesis. Something that describes the mapping between abstract counters for perf events and actual pfm counters.


Repository:
  rL LLVM

https://reviews.llvm.org/D45360





More information about the llvm-commits mailing list