[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