[PATCH] D98564: [AArch64] Peephole rule to remove redundant cmp after cset.

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 16 02:17:48 PDT 2021


dmgreen added a comment.

I imagine that the cset; cmp from the fp16 tests is just a missing optimization in ISel, and there might be good reason to do so there if it helps simplify the graph.

But like you said, this does fit in with the other peepholing code. And if it helps both ISels, that is good.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98564/new/

https://reviews.llvm.org/D98564



More information about the llvm-commits mailing list