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

[Vitaly Buka via llvm-commits]

Thanks, it works.

On Tue, 9 Nov 2021 at 16:09, David Blaikie via Phabricator <
reviews at reviews.llvm.org> wrote:

> dblaikie added a comment.
>
> In D111414#3120059 <https://reviews.llvm.org/D111414#3120059>,
> @vitalybuka wrote:
>
> > Looks like this patch
> https://lab.llvm.org/buildbot/#/builders/168/builds/3260/steps/10/logs/stdio
> >
> >   ==40944==ERROR: LeakSanitizer: detected memory leaks
> >   Direct leak of 1024 byte(s) in 1 object(s) allocated from:
> >       #0 0x387698 in malloc
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/compiler-rt/lib/asan/asan_malloc_linux.cpp:129:3
> >       #1 0x4f47d8 in initializeOutputBuffer
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/include/llvm/Demangle/Utility.h:199:31
> >       #2 0x4f47d8 in llvm::dlangDemangle(char const*)
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/lib/Demangle/DLangDemangle.cpp:29:8
> >       #3 0x3c9455 in DLangDemangleTestFixture::SetUp()
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/unittests/Demangle/DLangDemangleTest.cpp:20:39
> >       #4 0x5f710e in HandleExceptionsInMethodIfSupported<testing::Test,
> void>
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/src/gtest.cc
> >       #5 0x5f710e in testing::Test::Run()
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/src/gtest.cc:2503:3
> >       #6 0x5fbcdb in testing::TestInfo::Run()
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/src/gtest.cc:2684:11
> >       #7 0x5fd14f in testing::TestSuite::Run()
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/src/gtest.cc:2816:28
> >       #8 0x62dacd in testing::internal::UnitTestImpl::RunAllTests()
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/src/gtest.cc:5338:44
> >       #9 0x62bc2c in
> HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/src/gtest.cc
> >       #10 0x62bc2c in testing::UnitTest::Run()
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/src/gtest.cc:4925:10
> >       #11 0x5d95b0 in RUN_ALL_TESTS
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/googletest/include/gtest/gtest.h:2473:46
> >       #12 0x5d95b0 in main
> /b/sanitizer-x86_64-linux-bootstrap-asan/build/llvm-project/llvm/utils/unittest/UnitTestMain/TestMain.cpp:50:10
> >       #13 0x7f12de89a09a in __libc_start_main
> (/lib/x86_64-linux-gnu/libc.so.6+0x2409a)
> >   SUMMARY: AddressSanitizer: 1024 byte(s) leaked in 1 allocation(s).
>
> I don't have a sanitizer build setup currently, and couldn't reproduce
> this with valgrind (maybe used it incorrectly) - but hopefully
> 1bed03b5e3812dc6c659d285d77a2a577a2d113d <
> https://reviews.llvm.org/rG1bed03b5e3812dc6c659d285d77a2a577a2d113d>
> addresses this issue.
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D111414/new/
>
> https://reviews.llvm.org/D111414
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211109/71335d62/attachment.html>