[all-commits] [llvm/llvm-project] e77af7: [DebugInfo] Make DIArgList inherit from Metadata a...

Stephen Tozer via All-commits all-commits at lists.llvm.org
Fri Nov 17 06:05:09 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e77af7e1b07ac648c026922e4a0b07e9af35f714
      https://github.com/llvm/llvm-project/commit/e77af7e1b07ac648c026922e4a0b07e9af35f714
  Author: Stephen Tozer <stephen.tozer at sony.com>
  Date:   2023-11-17 (Fri, 17 Nov 2023)

  Changed paths:
    M llvm/include/llvm/AsmParser/LLParser.h
    M llvm/include/llvm/IR/DebugInfoMetadata.h
    M llvm/include/llvm/IR/Metadata.def
    M llvm/include/llvm/IR/Metadata.h
    M llvm/lib/AsmParser/LLParser.cpp
    M llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
    M llvm/lib/IR/AsmWriter.cpp
    M llvm/lib/IR/DebugInfoMetadata.cpp
    M llvm/lib/IR/LLVMContextImpl.cpp
    M llvm/lib/IR/LLVMContextImpl.h
    M llvm/lib/IR/Metadata.cpp
    M llvm/lib/IR/TypeFinder.cpp
    M llvm/lib/IR/Verifier.cpp

  Log Message:
  -----------
  [DebugInfo] Make DIArgList inherit from Metadata and always unique (#72147)

This patch changes the `DIArgList` class's inheritance from `MDNode` to
`Metadata, ReplaceableMetadataImpl`, and ensures that it is always
unique, i.e. a distinct DIArgList should never be produced.

This should not result in any changes to IR or bitcode parsing and
printing, as the format for DIArgList is unchanged, and the order in which it
appears should also be identical. As a minor note, this patch also fixes
a gap in the verifier, where the ValueAsMetadata operands to a DIArgList
would not be visited.




More information about the All-commits mailing list