[PATCH] D67403: [AArch64] MachineCombiner FMA matching. NFC.
Sebastian Pop via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 11 08:39:04 PDT 2019
sebpop added a comment.
Almost LGTM.
================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:3669
+ Found = Match(AArch64::FMULHrr, 1, MCP::FMULADDH_OP1);
+ Found = Match(AArch64::FMULHrr, 2, MCP::FMULADDH_OP2) ? true : Found;
break;
----------------
SjoerdMeijer wrote:
> sebpop wrote:
> > What about avoiding the cond_expr with an or_expr? Something like this:
> > ```
> > Found = Match(1);
> > Found |= Match(2);
> > ```
> Yeah, I like that, that's even better.
>
> Thanks for reviewing, I will make this change before committing.
I really meant here and below...
================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:3680
+ ? true
+ : Found;
+
----------------
... here and below, until the end of the patch, please replace cond_exprs with bit_or exprs. Thanks!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67403/new/
https://reviews.llvm.org/D67403
More information about the llvm-commits
mailing list