[llvm] r297029 - [InstSimplify] remove misleading comments; NFC

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 08:49:36 PST 2017


Author: spatel
Date: Mon Mar  6 10:49:35 2017
New Revision: 297029

URL: http://llvm.org/viewvc/llvm-project?rev=297029&view=rev
Log:
[InstSimplify] remove misleading comments; NFC

Div/rem-of-0 does not cause faults/undef (not the same as div/rem-by-0).

Modified:
    llvm/trunk/lib/Analysis/InstructionSimplify.cpp

Modified: llvm/trunk/lib/Analysis/InstructionSimplify.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/InstructionSimplify.cpp?rev=297029&r1=297028&r2=297029&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/InstructionSimplify.cpp (original)
+++ llvm/trunk/lib/Analysis/InstructionSimplify.cpp Mon Mar  6 10:49:35 2017
@@ -1031,7 +1031,7 @@ static Value *SimplifyDiv(Instruction::B
   if (match(Op0, m_Undef()))
     return Constant::getNullValue(Op0->getType());
 
-  // 0 / X -> 0, we don't need to preserve faults!
+  // 0 / X -> 0
   if (match(Op0, m_Zero()))
     return Op0;
 
@@ -1201,7 +1201,7 @@ static Value *SimplifyRem(Instruction::B
   if (match(Op0, m_Undef()))
     return Constant::getNullValue(Op0->getType());
 
-  // 0 % X -> 0, we don't need to preserve faults!
+  // 0 % X -> 0
   if (match(Op0, m_Zero()))
     return Op0;
 




More information about the llvm-commits mailing list