[llvm] r340059 - [X86] Use hasOneUse instead of isOnlyUserOf. NFCI
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 17 10:57:26 PDT 2018
Author: ctopper
Date: Fri Aug 17 10:57:25 2018
New Revision: 340059
URL: http://llvm.org/viewvc/llvm-project?rev=340059&view=rev
Log:
[X86] Use hasOneUse instead of isOnlyUserOf. NFCI
isOnlyUserOf is a little heavier because it allows the node to be used multiple times by the other node. In this case we are looking at a truncate which only has one operand so we know it can only use it once. Thus hasOneUse is better.
Modified:
llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=340059&r1=340058&r2=340059&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Fri Aug 17 10:57:25 2018
@@ -36780,7 +36780,7 @@ static SDValue combineTruncatedArithmeti
};
// Don't combine if the operation has other uses.
- if (!N->isOnlyUserOf(Src.getNode()))
+ if (!Src.hasOneUse())
return SDValue();
// Only support vector truncation for now.
More information about the llvm-commits
mailing list