aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Frontend/CreateInvocationFromCommandLine.cpp
diff options
context:
space:
mode:
authorAndrea Di Biagio <Andrea_DiBiagio@sn.scee.net>2018-11-12 13:09:39 +0000
committerAndrea Di Biagio <Andrea_DiBiagio@sn.scee.net>2018-11-12 13:09:39 +0000
commitdda90323141f38b278485488d52fa7a08c8f70e9 (patch)
tree656b76ff139a5c1de09dc9b2f47ed44260d25d78 /clang/lib/Frontend/CreateInvocationFromCommandLine.cpp
parente194c02849734d296da49d901ce3c66a06ec186b (diff)
downloadllvm-dda90323141f38b278485488d52fa7a08c8f70e9.zip
llvm-dda90323141f38b278485488d52fa7a08c8f70e9.tar.gz
llvm-dda90323141f38b278485488d52fa7a08c8f70e9.tar.bz2
[llvm-mca] Correctly update the resource strategy for processor resources with multiple units.
When looking at the tests committed by Roman at r346587, I noticed that numbers reported by the resource pressure for PdAGU01 were wrong. In particular, according to the aut-generated CHECK lines in tests memcpy-like-test.s and store-throughput.s, resource pressure for PdAGU01 was not uniformly distributed among the two AGEN pipes. It turns out that the reason why pressure was not correctly distributed, was because the "resource selection strategy" object associated with PdAGU01 was not correctly updated on the event of AGEN pipe used. As a result, llvm-mca was not simulating a round-robin pipeline allocation for PdAGU01. Instead, PdAGU1 was always prioritized over PdAGU0. This patch fixes the issue; now processor resource strategy objects for resources declaring multiple units, are correctly notified in the event of "resource used". llvm-svn: 346650
Diffstat (limited to 'clang/lib/Frontend/CreateInvocationFromCommandLine.cpp')
0 files changed, 0 insertions, 0 deletions