================ @@ -141,3 +141,9 @@ TEST_F(DWARFExpressionCompactPrinterTest, Test_OP_nop_OP_reg) { TEST_F(DWARFExpressionCompactPrinterTest, Test_OP_LLVM_nop_OP_reg) { TestExprPrinter({DW_OP_LLVM_user, DW_OP_LLVM_nop, DW_OP_reg0}, "R0"); } + +TEST_F(DWARFExpressionCompactPrinterTest, Test_OP_LLVM_user_unknown_subop) { + TestExprPrinter({DW_OP_LLVM_user, DW_OP_LLVM_form_aspace_address}, + "<unknown op DW_OP_LLVM_user (233) subop " + "DW_OP_LLVM_form_aspace_address (2)>"); ---------------- OCHyams wrote:
Oh right I understand now, yes that sentence feels a bit ambiguous to me. But I don't think it's worth revisiting, it's good enough for this utility function imo. https://github.com/llvm/llvm-project/pull/153883 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits