[all-commits] [llvm/llvm-project] fdf7e4: llvm-c: Add LLVMDeleteInstruction to fix a test issue

Nicolai Hähnle via All-commits all-commits at lists.llvm.org
Thu Jul 7 05:29:59 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fdf7e437bfc9df20454361a3453e2811be9da56c
      https://github.com/llvm/llvm-project/commit/fdf7e437bfc9df20454361a3453e2811be9da56c
  Author: Nicolai Hähnle <nicolai.haehnle at amd.com>
  Date:   2022-07-07 (Thu, 07 Jul 2022)

  Changed paths:
    M llvm/docs/ReleaseNotes.rst
    M llvm/include/llvm-c/Core.h
    M llvm/lib/IR/Core.cpp
    M llvm/tools/llvm-c-test/metadata.c

  Log Message:
  -----------
  llvm-c: Add LLVMDeleteInstruction to fix a test issue

Not deleting the loose instruction with metadata associated to it causes
an assertion when the LLVMContext is destroyed. This was previously
hidden by the fact that llvm-c-test does not call LLVMShutdown. The
planned removal of ManagedStatic exposed this issue.

Differential Revision: https://reviews.llvm.org/D129114


  Commit: 64a78c8501150cb1d65b1e9cc8804aea6186e0cb
      https://github.com/llvm/llvm-project/commit/64a78c8501150cb1d65b1e9cc8804aea6186e0cb
  Author: Nicolai Hähnle <nicolai.haehnle at amd.com>
  Date:   2022-07-07 (Thu, 07 Jul 2022)

  Changed paths:
    M llvm/lib/Target/X86/X86InstrFMA3Info.cpp
    M llvm/tools/bugpoint/bugpoint.cpp
    M llvm/tools/dsymutil/dsymutil.cpp
    M llvm/tools/lli/lli.cpp
    M llvm/tools/llvm-as/llvm-as.cpp
    M llvm/tools/llvm-cov/llvm-cov.cpp
    M llvm/tools/llvm-cvtres/llvm-cvtres.cpp
    M llvm/tools/llvm-mt/llvm-mt.cpp
    M llvm/tools/llvm-pdbutil/llvm-pdbutil.cpp
    M llvm/tools/llvm-rc/llvm-rc.cpp
    M llvm/unittests/Passes/PluginsTest.cpp

  Log Message:
  -----------
  Remove unnecessary includes of ManagedStatic.h

Differential Revision: https://reviews.llvm.org/D129115


Compare: https://github.com/llvm/llvm-project/compare/0f4339a835c0...64a78c850115


More information about the All-commits mailing list