Note: The constructor definition cannot be inline without some refactoring
as it introduces a circular dependency between the headers
llvm/IR/DebugInfoMetadata.h (this file) and llvm/IR/IntrinsicInst.h (where
DbgVariableIntrinsic is defined).
This is used by the assignment tracking patch stack (starts at D132220). I'm happy to wait for those patches to land before landing this, to avoid having unused code hanging around.
It seems like it could be const and still access everything you need here...?