[all-commits] [llvm/llvm-project] db394e: [RemoveDIs] Update DIBuilder C API with DbgRecord ...

[Carlos Alberto Enciso via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: db394edf95e7e47e2a11c570d65a8f5005af7849
      https://github.com/llvm/llvm-project/commit/db394edf95e7e47e2a11c570d65a8f5005af7849
  Author: Carlos Alberto Enciso <Carlos.Enciso at sony.com>
  Date:   2024-06-18 (Tue, 18 Jun 2024)

  Changed paths:
    M llvm/bindings/ocaml/debuginfo/debuginfo_ocaml.c
    M llvm/docs/ReleaseNotes.rst
    M llvm/docs/RemoveDIsDebugInfo.md
    M llvm/include/llvm-c/DebugInfo.h
    M llvm/lib/IR/DebugInfo.cpp
    R llvm/test/Bindings/llvm-c/debug_info.ll
    M llvm/test/Bindings/llvm-c/debug_info_new_format.ll
    M llvm/tools/llvm-c-test/debuginfo.c
    M llvm/tools/llvm-c-test/llvm-c-test.h
    M llvm/tools/llvm-c-test/main.c

  Log Message:
  -----------
  [RemoveDIs] Update DIBuilder C API with DbgRecord functions. (#95535)

The DIBuilder C API was changed to deal with DbgRecord functions:

https://github.com/llvm/llvm-project/pull/84915
https://github.com/llvm/llvm-project/pull/85657
https://github.com/llvm/llvm-project/pull/92417#issuecomment-2120078326

As discussed by @nikic and @OCHyams:

https://github.com/llvm/llvm-project/pull/92417#issuecomment-2144505440

> For the intrinsic-inserting functions, do you think we should:
>
> a) mark as deprecated but otherwise leave them alone for now.
> b) mark as deprecated and change their behaviour so that they
>    do nothing and return nullptr.
> c) outright delete them (it sounds like you're voting this one,
>    but just wanted to double check).

This patch implements option (c).



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications