From e51b57bee0f63a49e0a016256cda63535b88684c Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Mon, 8 Feb 2010 22:19:11 +0000 Subject: [PATCH] ConstantFoldConstantExpression can theoretically return the original expression; don't go into an infinite loop if it does. llvm-svn: 95591 --- llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp index 5f730e03c6f5..2fbd8dc098a8 100644 --- a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp +++ b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp @@ -892,8 +892,10 @@ static const MCExpr *LowerConstant(const Constant *CV, AsmPrinter &AP) { // If the code isn't optimized, there may be outstanding folding // opportunities. Attempt to fold the expression using TargetData as a // last resort before giving up. - if (Constant *C = ConstantFoldConstantExpression(CE, AP.TM.getTargetData())) - return LowerConstant(C, AP); + if (Constant *C = + ConstantFoldConstantExpression(CE, AP.TM.getTargetData())) + if (C != CE) + return LowerConstant(C, AP); #ifndef NDEBUG CE->dump(); #endif