[llvm] r257094 - Fix crash when printing instructions that have a metadata attached but no parent.
Mehdi Amini via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 7 12:14:30 PST 2016
Author: mehdi_amini
Date: Thu Jan 7 14:14:30 2016
New Revision: 257094
URL: http://llvm.org/viewvc/llvm-project?rev=257094&view=rev
Log:
Fix crash when printing instructions that have a metadata attached but no parent.
Fix PR24852 (crash with -debug -instcombine)
Patch by Than McIntosh <thanm at google.com>
Summary:
Add guards to the asm writer to prevent crashing
when dumping an instruction that has no basic
block.
Differential Revision: http://reviews.llvm.org/D15798
From: Than McIntosh <thanm at google.com>
Added:
llvm/trunk/unittests/IR/AsmWriterTest.cpp
Modified:
llvm/trunk/lib/IR/AsmWriter.cpp
llvm/trunk/unittests/IR/CMakeLists.txt
Modified: llvm/trunk/lib/IR/AsmWriter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/AsmWriter.cpp?rev=257094&r1=257093&r2=257094&view=diff
==============================================================================
--- llvm/trunk/lib/IR/AsmWriter.cpp (original)
+++ llvm/trunk/lib/IR/AsmWriter.cpp Thu Jan 7 14:14:30 2016
@@ -3121,7 +3121,7 @@ void AssemblyWriter::printMetadataAttach
return;
if (MDNames.empty())
- TheModule->getMDKindNames(MDNames);
+ MDs[0].second->getContext().getMDKindNames(MDNames);
for (const auto &I : MDs) {
unsigned Kind = I.first;
Added: llvm/trunk/unittests/IR/AsmWriterTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/IR/AsmWriterTest.cpp?rev=257094&view=auto
==============================================================================
--- llvm/trunk/unittests/IR/AsmWriterTest.cpp (added)
+++ llvm/trunk/unittests/IR/AsmWriterTest.cpp Thu Jan 7 14:14:30 2016
@@ -0,0 +1,37 @@
+//===- llvm/unittest/IR/AsmWriter.cpp - AsmWriter tests -------------------===//
+//
+// The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+#include "llvm/IR/IRBuilder.h"
+#include "llvm/IR/Function.h"
+#include "llvm/IR/LLVMContext.h"
+#include "llvm/IR/MDBuilder.h"
+#include "llvm/IR/Module.h"
+#include "gtest/gtest.h"
+
+using namespace llvm;
+
+namespace {
+
+TEST(AsmWriterTest, DebugPrintDetachedInstruction) {
+
+ // PR24852: Ensure that an instruction can be printed even when it
+ // has metadata attached but no parent.
+ LLVMContext Ctx;
+ auto Ty = Type::getInt32Ty(Ctx);
+ auto Undef = UndefValue::get(Ty);
+ std::unique_ptr<BinaryOperator> Add(BinaryOperator::CreateAdd(Undef, Undef));
+ Add->setMetadata(
+ "", MDNode::get(Ctx, {ConstantAsMetadata::get(ConstantInt::get(Ty, 1))}));
+ std::string S;
+ raw_string_ostream OS(S);
+ Add->print(OS);
+ std::size_t r = OS.str().find("<badref> = add i32 undef, undef, !<empty");
+ EXPECT_TRUE(r != std::string::npos);
+}
+
+}
Modified: llvm/trunk/unittests/IR/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/IR/CMakeLists.txt?rev=257094&r1=257093&r2=257094&view=diff
==============================================================================
--- llvm/trunk/unittests/IR/CMakeLists.txt (original)
+++ llvm/trunk/unittests/IR/CMakeLists.txt Thu Jan 7 14:14:30 2016
@@ -6,6 +6,7 @@ set(LLVM_LINK_COMPONENTS
)
set(IRSources
+ AsmWriterTest.cpp
AttributesTest.cpp
ConstantRangeTest.cpp
ConstantsTest.cpp
More information about the llvm-commits
mailing list