Description

Adds a few utility functions around debug information and changes instruction operand handling to support debug metadata, with some fixes to error messaging around value types too.

Changes

• Changes InstructionValue::get_operand family of functions (including get_operands) to return a new OperandValue. They currently had Either<BasicValue, BasicBlock> which is insufficient for operands - they can be metadata values as well.
• Added Helper functions for the transitional DbgRecord changes.
  Module::is_new_debug_format - this corresponds to LLVMIsNewDbgInfoFormat
Module::set_new_debug_format - this corresponds to LLVMSetIsNewDbgInfoFormat
• Added as_metadata_value to the suite of DI* types, as all these types are metadata values that can be fetched via instruction operands (old) or DbgRecord (new)
• Better output to the panics in the enum variants, so you can see what kind was unimplemente
• Updated tests to all pass for the new get_operand call.

How This Has Been Tested

This was tested with llvm 20.1 locally in a personal project. Ex usage such as:

    module.set_new_debug_format(false);
    module.get_functions().for_each(|function| {
        function.get_basic_blocks().iter().for_each(|block| {
            block.get_instructions().skip(1).for_each(|instr| {
                    (0..instr.get_num_operands()).for_each(|i| {
                            if let Some(OperandValue::BasicMetadataValue(metadata)) =
                                instr.get_operand(i)
                            {
                                ... Handle debug metadata operands here....
                            }
                        });
                    });
                }
            });
        });

Breaking Changes

Changes InstructionValue::get_operand and OperandIter to use OperandValue instead of Either<BasicValue, BasicBlock>. This was required because operands on a value instruction can be more than just BasicValue or BasicBlock, they can also be MetadataValue in the case of debug intrinsics.

• All left() calls must now be into_basic_value() or try_into() or into()
• All right() calls must now be into_basic_block() or try_into() or into()

Checklist

• I have read the Contributing Guide