[PATCH] D98174: [MCA] Add tests for IPC on Cortex-A55
Andrew Savonichev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 9 07:00:56 PST 2021
asavonic added a comment.
In D98174#2613867 <https://reviews.llvm.org/D98174#2613867>, @andreadb wrote:
> One last question.
> What is the plan with the SDIV test cases? I don't think that there is anything that we can do to improve that simulation, since it would require knowledge that isn't available at simulation time. The risk is to end up with a test which isn't very useful in practice (it will always be marked as XFAIL). In which case, I suggest to remove those DIV tests entirely.
They are not very useful as "tests", I agree, but they can be useful as a documentation,
highlighting the cases where MCA and hardware do not match. Although there is no point
in having two tests for the same issue, so we can remove one of them.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98174/new/
https://reviews.llvm.org/D98174
More information about the llvm-commits
mailing list