[clang-tools-extra] f325b5b - [clang-tidy] Support std::string_view in readability-redundant-string-cstr

2023-01-02 Thread Tamas Berghammer via cfe-commits
Author: Tamas Berghammer Date: 2023-01-02T06:24:00-05:00 New Revision: f325b5b8cb1d6c16dd04c88380caea7b0a7750eb URL: https://github.com/llvm/llvm-project/commit/f325b5b8cb1d6c16dd04c88380caea7b0a7750eb DIFF: https://github.com/llvm/llvm-project/commit/f325b5b8cb1d6c16dd04c88380caea7b0a7750eb.di

[libcxxabi] r303737 - __cxa_demangle: Fix constructor cv qualifier handling

2017-05-24 Thread Tamas Berghammer via cfe-commits
Author: tberghammer Date: Wed May 24 06:21:34 2017 New Revision: 303737 URL: http://llvm.org/viewvc/llvm-project?rev=303737&view=rev Log: __cxa_demangle: Fix constructor cv qualifier handling Summary: Previously if we parsed a constructor then we set parsed_ctor_dtor_cv to true and never reseted

Re: [PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

2015-12-22 Thread Tamas Berghammer via cfe-commits
tberghammer abandoned this revision. tberghammer added a comment. Thank you for adding the new debugger tuning feature. I am using LLDB so "-glldb" will work for me perfectly. There is one thing we might consider is that the accelerator tables increasing the size of the binary by ~10% (measured

Re: [PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

2015-11-10 Thread Tamas Berghammer via cfe-commits
tberghammer added a comment. I am not in a hurry with getting this change in, but happy to take a look for enabling the tuning if somebody can explain me what you mean under it (I have almost no llvm/clang development experience) http://reviews.llvm.org/D14354 __

Re: [PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

2015-11-06 Thread Tamas Berghammer via cfe-commits
tberghammer added a comment. If you plan to do the "tuning" in clang in the near future then we can abandon this change in favor of that one Comment at: lib/Driver/Tools.cpp:3880 @@ +3879,3 @@ + // -gdwarf-accel-tables should turn on -g and enable the genereation of the + //

Re: [PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

2015-11-05 Thread Tamas Berghammer via cfe-commits
tberghammer added a comment. In http://reviews.llvm.org/D14354#282870, @probinson wrote: > So, currently you get accel tables by default for platforms that "tune" for > LLDB, currently Darwin and FreeBSD. > Are you wanting to use LLDB on another platform, or did you want accel > tables for a d

Re: [PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

2015-11-05 Thread Tamas Berghammer via cfe-commits
2015 at 8:17 AM Saleem Abdulrasool wrote: > > > On Wednesday, November 4, 2015, Tamas Berghammer via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> tberghammer created this revision. >> tberghammer added a reviewer: echristo. >> tberghammer ad

Re: [PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

2015-11-05 Thread Tamas Berghammer via cfe-commits
tberghammer updated this revision to Diff 39385. tberghammer added a comment. Add a test (I have no experience writing clang tests so please let me know if I approach is wrong) http://reviews.llvm.org/D14354 Files: include/clang/Driver/Options.td lib/Driver/Tools.cpp test/Driver/dwarf-ac

[PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

2015-11-04 Thread Tamas Berghammer via cfe-commits
tberghammer created this revision. tberghammer added a reviewer: echristo. tberghammer added a subscriber: cfe-commits. Add new compiler flag to enable the generation of dwarf accelerator tables The dwarf accelerator tables already generated on darwin platforms. This CL ands a new flag to clang