[PATCH] D71701: [AArch64] Peephole optimization. Merge AND and TST instructions
Sjoerd Meijer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 10 03:32:41 PST 2020
SjoerdMeijer added a comment.
> So, do I need to create another patch with ISel implementation (I mean - create another differential revision), or it is better to change this patch?
As is usually the case, there are multiple ways to achieve the same thing.... We don't need 2 implementations, but just pick the right approach. Dave suggested isel, and is open to alternatives. This patch is small and simple, and so I think this is a good place for it (as also remarked by Dave). And @craig.topper suggestion is very interesting, that will probably involve the least amount of code if something similar works for us, so that's probably best to check that first.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71701/new/
https://reviews.llvm.org/D71701
More information about the llvm-commits
mailing list