From 244d2bdb24bc24c957d470714fd69f62a0c68127 Mon Sep 17 00:00:00 2001 From: Dale Johannesen Date: Tue, 27 Apr 2010 02:10:05 +0000 Subject: [PATCH] Revert a small part of 102372; this fixes at least one of the dbg testsuite regressions. I don't think this is really the right fix; this change exposed an existing problem upstream somewhere. llvm-svn: 102410 --- lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp index d663521abaf..9d239c1bd7c 100644 --- a/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp +++ b/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp @@ -3834,10 +3834,15 @@ SelectionDAGBuilder::visitIntrinsicCall(const CallInst &I, unsigned Intrinsic) { return 0; DAG.AddDbgValue(SDV, N.getNode(), isParameter); } else { + // Generating Undefs here seems to be actively harmful because it + // affects the line numbers. + return 0; +#if 0 // This isn't useful, but it shows what we're missing. SDV = DAG.getDbgValue(Variable, UndefValue::get(Address->getType()), 0, dl, SDNodeOrder); DAG.AddDbgValue(SDV, 0, isParameter); +#endif } return 0; }