[all-commits] [llvm/llvm-project] f2df4b: [AsmParser] Support non-consecutive global value n...

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f2df4bfe54cd4161b7a2d89a0bc5dc69e747b01e
      https://github.com/llvm/llvm-project/commit/f2df4bfe54cd4161b7a2d89a0bc5dc69e747b01e
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2024-01-31 (Wed, 31 Jan 2024)

  Changed paths:
    M llvm/include/llvm/AsmParser/LLParser.h
    A llvm/include/llvm/AsmParser/NumberedValues.h
    M llvm/include/llvm/AsmParser/SlotMapping.h
    M llvm/lib/AsmParser/LLParser.cpp
    M llvm/lib/CodeGen/MIRParser/MIParser.cpp
    A llvm/test/Assembler/skip-value-numbers-globals.ll
    M llvm/test/Assembler/skip-value-numbers-invalid.ll
    M llvm/unittests/AsmParser/AsmParserTest.cpp

  Log Message:
  -----------
  [AsmParser] Support non-consecutive global value numbers (#80013)

https://github.com/llvm/llvm-project/pull/78171 added support for
non-consecutive local value numbers. This extends the support for global
value numbers (for globals and functions).

This means that it is now possible to delete an unnamed global
definition/declaration without breaking the IR.

This is a lot less common than unnamed local values, but it seems like
something we should support for consistency. (Unnamed globals are used a
lot in Rust though.)