[PowerPC] Revert r314666.
authorTim Shen <timshen91@gmail.com>
Mon, 2 Oct 2017 23:20:06 +0000 (23:20 +0000)
committerTim Shen <timshen91@gmail.com>
Mon, 2 Oct 2017 23:20:06 +0000 (23:20 +0000)
See https://reviews.llvm.org/D38172.

I tried to XFAIL it, but sometimes XPASS triggers the bot. Simply
revert it.

llvm-svn: 314739

llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
llvm/test/CodeGen/PowerPC/debuginfo-split-int.ll [deleted file]

index f331fc7..f76363a 100644 (file)
@@ -864,13 +864,8 @@ void DAGTypeLegalizer::SetExpandedInteger(SDValue Op, SDValue Lo,
   AnalyzeNewValue(Hi);
 
   // Transfer debug values.
-  if (DAG.getDataLayout().isBigEndian()) {
-    transferDbgValues(DAG, Op, Hi, 0);
-    transferDbgValues(DAG, Op, Lo, Hi.getValueSizeInBits());
-  } else {
-    transferDbgValues(DAG, Op, Lo, 0);
-    transferDbgValues(DAG, Op, Hi, Lo.getValueSizeInBits());
-  }
+  transferDbgValues(DAG, Op, Lo, 0);
+  transferDbgValues(DAG, Op, Hi, Lo.getValueSizeInBits());
 
   // Remember that this is the result of the node.
   std::pair<SDValue, SDValue> &Entry = ExpandedIntegers[Op];
diff --git a/llvm/test/CodeGen/PowerPC/debuginfo-split-int.ll b/llvm/test/CodeGen/PowerPC/debuginfo-split-int.ll
deleted file mode 100644 (file)
index 42279ca..0000000
+++ /dev/null
@@ -1,68 +0,0 @@
-; RUN: llc < %s -stop-after=isel -o - | FileCheck %s
-; FIXME: See the comments in D38172, regarding the name "isel" is registered only by AMDGPU.
-; XFAIL: *
-
-source_filename = "foo.c"
-target datalayout = "E-m:e-p:32:32-i64:64-n32"
-target triple = "ppc32"
-
-; Verify that, when handling split-up integers, the
-; transferring of debug info takes the endianness
-; into consideration.
-;
-; The fragment expression at offset 0 should correspond
-; to the high part of the value on big-endian targets.
-
-; This basis of this ll file was created by running:
-;   clang --target=powerpc -O1 -S -g -emit-llvm foo.c
-;
-; with foo.c being the program:
-;   unsigned long long foo(void);
-;   void bar() {
-;     volatile unsigned long long result = foo();
-;   }
-;
-; This file is a slight tweak of that output, with irrelevant
-; lifetime intrinsics, metadata, and debug info being removed.
-
-; CHECK: [[DL:![0-9]+]] = !DILocalVariable(name: "result"
-;
-; High 32 bits in R3, low 32 bits in R4
-; CHECK: %0 = COPY %r3
-; CHECK: DBG_VALUE debug-use %0, debug-use _, [[DL]], !DIExpression(DW_OP_LLVM_fragment, 0, 32)
-; CHECK: %1 = COPY %r4
-; CHECK: DBG_VALUE debug-use %1, debug-use _, [[DL]], !DIExpression(DW_OP_LLVM_fragment, 32, 32)
-define void @bar() local_unnamed_addr #0 !dbg !6 {
-  %1 = alloca i64, align 8
-  %2 = tail call i64 @foo()
-  tail call void @llvm.dbg.value(metadata i64 %2, metadata !10, metadata !DIExpression()), !dbg !13
-  store volatile i64 %2, i64* %1, align 8
-  ret void
-}
-
-declare i64 @foo() local_unnamed_addr
-
-; Function Attrs: nounwind readnone speculatable
-declare void @llvm.dbg.value(metadata, metadata, metadata) #1
-
-attributes #0 = { nounwind }
-attributes #1 = { nounwind readnone speculatable }
-
-!llvm.dbg.cu = !{!0}
-!llvm.module.flags = !{!3, !4}
-!llvm.ident = !{!5}
-
-!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang version 6.0.0", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2)
-!1 = !DIFile(filename: "foo.c", directory: "/")
-!2 = !{}
-!3 = !{i32 2, !"Dwarf Version", i32 4}
-!4 = !{i32 2, !"Debug Info Version", i32 3}
-!5 = !{!"clang version 6.0.0"}
-!6 = distinct !DISubprogram(name: "bar", scope: !1, file: !1, line: 2, type: !7, isLocal: false, isDefinition: true, scopeLine: 2, isOptimized: true, unit: !0, variables: !9)
-!7 = !DISubroutineType(types: !8)
-!8 = !{null}
-!9 = !{!10}
-!10 = !DILocalVariable(name: "result", scope: !6, file: !1, line: 3, type: !11)
-!11 = !DIDerivedType(tag: DW_TAG_volatile_type, baseType: !12)
-!12 = !DIBasicType(name: "long long unsigned int", size: 64, encoding: DW_ATE_unsigned)
-!13 = !DILocation(line: 3, column: 31, scope: !6)