[PATCH] D48676: [Local] Teach insertReplacementDbgValues basic integer/pointer conversions

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added inline comments.


================
Comment at: include/llvm/IR/DebugInfoMetadata.h:785
+  /// signed nor unsigned.
+  Optional<Signedness> getSignedness() const;
+
----------------
Why not make an `enum { Unknown, Signed, Unsigned }` ?


================
Comment at: test/Transforms/InstCombine/cast-set-preserve-signed-dbg-val.ll:17
+  ; CHECK-NEXT: call void @llvm.dbg.value(metadata i16 [[and]], metadata [[C:![0-9]+]],
+  ; CHECK-SAME:    metadata !DIExpression(DW_OP_constu, 15, DW_OP_shr, DW_OP_constu, 4294967295, DW_OP_mul, DW_OP_stack_value, DW_OP_LLVM_fragment, 16, 16))
+  ;
----------------
wouldn't `DW_OP_lit0 DW_OP_not` be shorter than `DW_OP_constu 0xfffffff` ?


https://reviews.llvm.org/D48676