[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