[all-commits] [llvm/llvm-project] 4aa776: [bazel] Restore libpfm as a conditional dependency...
Jordan Rupprecht via All-commits
all-commits at lists.llvm.org
Wed Dec 28 08:13:35 PST 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 4aa77690b98e564c598505749d4eb57b9a8be32c
https://github.com/llvm/llvm-project/commit/4aa77690b98e564c598505749d4eb57b9a8be32c
Author: Jordan Rupprecht <rupprecht at google.com>
Date: 2022-12-28 (Wed, 28 Dec 2022)
Changed paths:
M utils/bazel/.bazelrc
M utils/bazel/WORKSPACE
M utils/bazel/llvm-project-overlay/llvm/BUILD.bazel
A utils/bazel/third_party_build/pfm.BUILD
Log Message:
-----------
[bazel] Restore libpfm as a conditional dependency for exegesis.
We used to have `pfm` built into exegesis, although since it's an external dependency we marked it as a manual target. Because of this we didn't have buildbot coverage and so we removed it in D134510 after we had a few breakages that weren't caught. This adds it back, but with three possible states similar to the story with `mpfr`, i.e. it can either be disabled, built from external sources (git/make), or use whatever `-lpfm` is installed on the system.
This change is modeled after D119547. Like that patch, the default is off (matching the status quo), but unlike that patch we don't enable it for CI because IIRC we don't have the package installed there, and building from source might be expensive. We could enable it later either after installing it on buildbot machines or by measuring build cost and deeming it OK.
Reviewed By: GMNGeoffrey
Differential Revision: https://reviews.llvm.org/D138470
More information about the All-commits
mailing list