DIBuilder: Remove dead code
authorDuncan P. N. Exon Smith <dexonsmith@apple.com>
Wed, 1 Oct 2014 21:14:20 +0000 (21:14 +0000)
committerDuncan P. N. Exon Smith <dexonsmith@apple.com>
Wed, 1 Oct 2014 21:14:20 +0000 (21:14 +0000)
I neglected to update `DIBuilder::createPieceExpression()` in r218797,
which I noticed while rebasing a patch for PR17891.  On closer
inspection, it looks like dead code.

If there are any downstream users of this, you should transition to the
more general `createExpression()`.  Or, we can add this back, but then
it should just forward to `createExpression()`.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@218820 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/IR/DIBuilder.h
lib/IR/DIBuilder.cpp

index d12a0c609db40b64056225385a89efe9f46f7cb6..3e9f43ad8a275df4a64417f498cb1fdc26736b6c 100644 (file)
@@ -505,14 +505,6 @@ namespace llvm {
     /// @param Addr        An array of complex address operations.
     DIExpression createExpression(ArrayRef<int64_t> Addr = None);
 
-    /// createPieceExpression - Create a descriptor to describe one part
-    /// of aggregate variable that is fragmented across multiple Values.
-    ///
-    /// @param OffsetInBytes Offset of the piece in bytes.
-    /// @param SizeInBytes   Size of the piece in bytes.
-    DIExpression createPieceExpression(unsigned OffsetInBytes,
-                                       unsigned SizeInBytes);
-
     /// createFunction - Create a new descriptor for the specified subprogram.
     /// See comments in DISubprogram for descriptions of these fields.
     /// @param Scope         Function scope.
index f1d4a21bb73ba640bdc9583bedb4c84902960a9d..743886946fc5fe1b01dc8f2ad79ce94dca79bb45 100644 (file)
@@ -1056,20 +1056,6 @@ DIExpression DIBuilder::createExpression(ArrayRef<int64_t> Addr) {
   return DIExpression(MDNode::get(VMContext, Elts));
 }
 
-/// createVariablePiece - Create a descriptor to describe one part
-/// of aggregate variable that is fragmented across multiple Values.
-DIExpression DIBuilder::createPieceExpression(unsigned OffsetInBytes,
-                                              unsigned SizeInBytes) {
-  assert(SizeInBytes > 0 && "zero-size piece");
-  Value *Addr[] = {
-      GetTagConstant(VMContext, DW_TAG_expression),
-      ConstantInt::get(Type::getInt64Ty(VMContext), dwarf::DW_OP_piece),
-      ConstantInt::get(Type::getInt64Ty(VMContext), OffsetInBytes),
-      ConstantInt::get(Type::getInt64Ty(VMContext), SizeInBytes)};
-
-  return DIExpression(MDNode::get(VMContext, Addr));
-}
-
 /// createFunction - Create a new descriptor for the specified function.
 /// FIXME: this is added for dragonegg. Once we update dragonegg
 /// to call resolve function, this will be removed.