diff options
| author | Andrea Di Biagio <andrea.dibiagio@sony.com> | 2020-05-10 18:55:58 +0100 |
|---|---|---|
| committer | Andrea Di Biagio <andrea.dibiagio@sony.com> | 2020-05-10 19:25:54 +0100 |
| commit | 47b95d7cf462b824b2fbea1eb1a64e9c3eaa1a95 (patch) | |
| tree | 29bf90646882967f092be9ebaaea0fd9802e3b07 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 93d5ae3af18f3e5d4232510274d59c1b4e5b8e77 (diff) | |
| download | llvm-47b95d7cf462b824b2fbea1eb1a64e9c3eaa1a95.zip llvm-47b95d7cf462b824b2fbea1eb1a64e9c3eaa1a95.tar.gz llvm-47b95d7cf462b824b2fbea1eb1a64e9c3eaa1a95.tar.bz2 | |
[MCA][InstrBuilder] Correctly mark reserved resources in initializeUsedResources.
This fixes a bug reported by Alex Renda on LLVMDev where mca did not correctly
mark a resource group as "reserved".
(See http://lists.llvm.org/pipermail/llvm-dev/2020-May/141485.html).
The issue was caused by a wrong check in function `initializeUsedResources`.
As a consequence of this, a resource group was left unreserved, and its field
`NumUnits` incorrectly reported an unrealistic number of consumed resource
units.
This patch fixes the issue with the handling of reserved resources in the
InstrBuilder class, and adds a simple test for it. Ideally, as suggested by
Andy Trick, most of these problems will disappear if in the future we will
introduce a (optional) DelayCycles vector for SchedWriteRes.
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions