[llvm] [RemoveDIs][NFC] Introduce DbgRecord base class [1/3] (PR #78252)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 20 03:33:02 PST 2024


https://github.com/OCHyams commented:

Thanks @jmorse  - I've now rebased and addressed comments.

After recent offline discussion we concluded that the 2nd patch,
>   2. Rename DPValue -> DbgVariableRecord, DPMarker -> DbgMarker." 

would be better off delayed - it can land with incoming documentation updates that @SLTozer is working on (unfortunately the existing patch became stale very quickly). That way we get functionality in an tested more quickly (and reverts are easier).

That means the next patch in this series will add the DbgLabel class (which I will rename DPLabel to keep things consistent until full rename is complete).

Does that plan sound acceptable?

https://github.com/llvm/llvm-project/pull/78252


More information about the llvm-commits mailing list