]> granicus.if.org Git - llvm/commit
[llvm-exegesis] Finish plumbing the `Config` field.
authorClement Courbet <courbet@google.com>
Tue, 8 Oct 2019 09:06:48 +0000 (09:06 +0000)
committerClement Courbet <courbet@google.com>
Tue, 8 Oct 2019 09:06:48 +0000 (09:06 +0000)
commit121dd37fe5bfd484164b17f2cb25e32800264de3
treeea03636c8b26f63e08d99ed5b5acdc8595cbc93c
parent8ce58b98ecfe56d33fc7f9b95d562fb42b7391fc
[llvm-exegesis] Finish plumbing the `Config` field.

Summary:
Right now there are no snippet generators that emit the `Config` Field,
but I plan to add it to investigate LEA operands for PR32326.

What was broken was:
 - `Config` Was not propagated up until the BenchmarkResult::Key.
 - Clustering should really consider different configs as measuring
 different things, so we should stabilize on (Opcode, Config) instead of
 just Opcode.

Reviewers: gchatelet

Subscribers: tschuett, llvm-commits, lebedev.ri

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D68629

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@374031 91177308-0d34-0410-b5e6-96231b3b80d8
test/tools/llvm-exegesis/X86/analysis-cluster-stabilization-config.test
tools/llvm-exegesis/lib/BenchmarkCode.h
tools/llvm-exegesis/lib/BenchmarkResult.h
tools/llvm-exegesis/lib/BenchmarkRunner.cpp
tools/llvm-exegesis/lib/Clustering.cpp
tools/llvm-exegesis/lib/CodeTemplate.h
tools/llvm-exegesis/lib/SnippetFile.cpp
tools/llvm-exegesis/lib/SnippetGenerator.cpp
unittests/tools/llvm-exegesis/X86/SnippetFileTest.cpp