[PATCH] D111414: [Demangle] Add minimal support for D programming language

Luís Ferreira via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 3 06:18:48 PDT 2021


ljmf00 added a comment.

In D111414#3105658 <https://reviews.llvm.org/D111414#3105658>, @jhenderson wrote:

> LGTM, modulo the previously-discussed licensing question - have you heard anything from the LLVM foundation board yet?

Updated the last comment. About licensing, I wrote to `llvm-dev` about that (here: https://lists.llvm.org/pipermail/llvm-dev/2021-October/153267.html ). Iain, the primary author of the code also wrote to the first patch. More context here https://reviews.llvm.org/D110576 . I don't know what is the next step in that regard if anything more is needed.



================
Comment at: llvm/lib/Demangle/DLangDemangle.cpp:39
+
+  // OutputBuffer internal buffer is not null terminated and therefore we need
+  // to add it to comply with C null terminated strings.
----------------
jhenderson wrote:
> 
Patch updated.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111414/new/

https://reviews.llvm.org/D111414



More information about the llvm-commits mailing list