User Details
- User Since
- Sep 9 2014, 6:00 AM (414 w, 1 d)
Thu, Aug 11
Noting related review: https://reviews.llvm.org/D115815 which added this variable to support this for other "external projects".
Feb 15 2022
Feb 9 2022
Feb 1 2022
Jan 31 2022
Thanks for the patch. When I originally added the multi-threaded support for the time profiler I considered adding the initialize/finish thread into the threadpool itself (llvm/lib/Support/ThreadPool.cpp). In the end I thought it was a bit messy there so just added it into LTO.cpp. With the increased use of multi-threading I think it would be worth reconsidering putting this into ThreadPool.cpp so we don't need to add this everywhere we want to trace multithreaded code. I think that the RAII code you propose will help this fit into ThreadPool.cpp neatly. For all I know there may be other problems with doing that, but I think it will be neater now this is used in more places. What do you think about trying that?
Mar 10 2021
With an additional change from sizeof(buf) to buf.size() this fixes the check-asan and check-ubsan tests.
Mar 9 2021
Mar 1 2021
Update with suggested changes to MinGW.cpp
Feb 27 2021
Feb 26 2021
Added comment on AllocatorSize.
Feb 9 2021
Feb 8 2021
Feb 5 2021
Add documentation to CMake.rst
I believe that the MCJIT failures are due to bug https:/bugs.llvm.org/show_bug.cgi?id=24978 rather than a problem in the Scudo port so I have added details of how I hit it to that bugzilla and opened two reviews to get this landed:
https://reviews.llvm.org/D96120 - [scudo] Port scudo sanitizer to Windows
https://reviews.llvm.org/D96133 - Allow building with scudo memory allocator on Windows
Feb 3 2021
I've focussed on the test test-global-init-nonzero-sm-pic.ll which fails writing to an address which (I believe) should be in the .data section but isn't.
Feb 2 2021
I managed to get this to fail in the debugger (for the cross-module-sm-pic-a.ll test):
Remove -fsanitize=scudo support for Windows in LLVM cmake, using LLVM_INTEGRATED_CRT_ALLOC instead.
Remove scudo_cxx from LLVM_INTEGRATED_CRT_ALLOC.
Dec 18 2020
Apologies for the delay, I've had other things taking my time.
Nov 9 2020
Resigning as reviewer as I also work for Sony.
Nov 5 2020
Oct 28 2020
Apologies for the delay, I've had a few other things on.
Sep 23 2020
Sep 22 2020
I'm going to concentrate on the standalone port as I think that's the way forward. If I get that working then can work through the other issues.
Sep 17 2020
Sep 15 2020
One small comment. Apart from that, LGTM.
Sep 14 2020
Re-upload patch with G LLVM Github Monorepo set.
Sep 11 2020
Is there a good sanity check that it is actually using Scudo rather than silently using the standard alloc?
Fixup scudo (sanitizer based) to work on Windows.
Aug 28 2020
Thanks for catching that.
Aug 27 2020
Fix llvm-config test.
I assume something in lit is parsing and recognizing this? Should remove that too?
Hi Matt,
I'd also like to see how they (currently supported allocators) compare vs. SCUDO and vs. tcmalloc3, if someone is willing to port those on Windows.
I've applied this to the monorepo and made a few more changes to allow it to build LLVM on Windows here: https://reviews.llvm.org/D86694
Aug 4 2020
Jul 31 2020
Thanks for your work on this. I've just tried out this version with rpmalloc (1.4.0) checked out out at the top of llvm-project.
Fixed comments and added test.
Jul 30 2020
Jul 28 2020
It looks like is causing one of the debuginfo-tests: llgdb-tests/nrvo-string.cpp to fail, run on Linux. Failure as below. I don't think the debuginfo-tests are run on any bot (but probably should be!). I bisected the failure back to this change.
Jul 23 2020
Jul 1 2020
Jun 30 2020
I notice that the improvements to rpmalloc for LLVM are still under discussion (https://github.com/mjansson/rpmalloc/issues/150) and targetted at rpmalloc v1.4.1.
Jun 23 2020
Jun 4 2020
May 29 2020
May 28 2020
Hi Sam,
May 22 2020
May 15 2020
May 14 2020
Removed excess dashes in man page. Checked with:
Change to double dashes.
May 13 2020
May 12 2020
Apr 22 2020
LGTM, with a few small comments.
Apr 15 2020
LGTM, with assertion messages adjusted.
Apr 14 2020
Would be nice if the test could check that the pid was being set to something but not sure how you could do that (beyond checking that it's a number).
Mar 24 2020
LGTM, thanks.