[PATCH] D147431: [Assignment Tracking][SROA] Handle createFragmentExpression failure

[Stephen Tozer via Phabricator via llvm-commits]

StephenTozer added inline comments.


================
Comment at: llvm/test/DebugInfo/Generic/assignment-tracking/sroa/fail-fragment.ll:11
+; CHECK: call void @llvm.dbg.value(metadata i32 poison, metadata ![[#]], metadata !DIExpression(DW_OP_LLVM_fragment, 0, 32))
+; CHECK: call void @llvm.dbg.value(metadata i32 %{{.*}}, metadata ![[#]], metadata !DIExpression(DW_OP_LLVM_fragment, 32, 32))
+
----------------
I'm curious - why is this value //not// poison? I'm assuming it's the upper fragment for the dbg.assign that couldn't be split, so I'd have thought it would be a kill location as well.


================
Comment at: llvm/test/DebugInfo/Generic/assignment-tracking/sroa/fail-fragment.ll:14-15
+; CHECK: if.else:
+; CHECK: dbg.value(metadata i32 2, metadata ![[#]], metadata !DIExpression(DW_OP_LLVM_fragment, 0, 32))
+; CHECK: dbg.value(metadata i32 0, metadata ![[#]], metadata !DIExpression(DW_OP_LLVM_fragment, 32, 32))
+
----------------
Tiniest nit.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147431/new/

https://reviews.llvm.org/D147431