[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