[PATCH] D97938: [AMDGPU][GlobalISel] Transform (fadd (fma x, y, (fmul u, v)), z) -> (fma x, y, (fma u, v, z))
Mirko Brkusanin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 3 10:49:31 PDT 2021
- Previous message: [PATCH] D97937: [AMDGPU][GlobalISel] Transform (fadd (fpext (fmul x, y)), z) -> (fma (fpext x), (fpext y), z)
- Next message: [PATCH] D98047: [AMDGPU][GlobalISel] Transform (fadd (fma x, y, (fpext (fmul u, v))), z) -> (fma x, y, (fma (fpext u), (fpext v), z))
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
mbrkusanin accepted this revision.
mbrkusanin added a comment.
This revision is now accepted and ready to land.
LGTM (with a nit)
================
Comment at: llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp:4589-4593
+ if (std::distance(MRI.use_instr_nodbg_begin(LHS->getOperand(0).getReg()),
+ MRI.use_instr_nodbg_end()) >
+ std::distance(MRI.use_instr_nodbg_begin(RHS->getOperand(0).getReg()),
+ MRI.use_instr_nodbg_end()))
+ std::swap(LHS, RHS);
----------------
use hasMoreUses
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97938/new/
https://reviews.llvm.org/D97938
- Previous message: [PATCH] D97937: [AMDGPU][GlobalISel] Transform (fadd (fpext (fmul x, y)), z) -> (fma (fpext x), (fpext y), z)
- Next message: [PATCH] D98047: [AMDGPU][GlobalISel] Transform (fadd (fma x, y, (fpext (fmul u, v))), z) -> (fma x, y, (fma (fpext u), (fpext v), z))
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
More information about the llvm-commits
mailing list