[PATCH] D143713: [ARM] Fix Chain/Glue Bug in PerformVMOVhrCombine
Sam Elliott via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 20 09:32:59 PST 2023
lenary added a comment.
In D143713#4139346 <https://reviews.llvm.org/D143713#4139346>, @dmgreen wrote:
>> Do you mean the AES pass tests? Do you think that should happen before we land this, they've been an issue for a little while.
>
> I think I just meant https://reviews.llvm.org/D143712#4117617 for "clear up the tests". A Multiple uses test would be useful to check that nothing odd goes on when the returned value is used elsewhere too.
Ah, right. Sorry, that leaked out my memory. Yes, I do intend to do a multiple uses test.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143713/new/
https://reviews.llvm.org/D143713
More information about the llvm-commits
mailing list