In D90972#2580020, @njames93 wrote:In D90972#2446448, @Flow wrote:In D90972#2394516, @JonasToth wrote:LGTM! thanks for fixing.
I am unable to commit this myself. The latest version of this change can be found in the install-run-clang-tidy-into-bin on my LLVM fork.
Noticed this is a little stale, Do you still want this landing?
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Mon, Feb 22
Mon, Feb 22
Dec 10 2020
Dec 10 2020
In D92490#2443573, @eugenis wrote:LGTM
Dec 10 2020, 10:21 AM · Restricted Project
In D90972#2394516, @JonasToth wrote:LGTM! thanks for fixing.
Dec 2 2020
Dec 2 2020
In D92490#2429404, @vitalybuka wrote:What is going to be the default?
Dec 2 2020, 11:07 PM · Restricted Project
Flow added reviewers for D92490: [msan] Do not use 77 as exit code, instead use 1: vitalybuka, eugenis.
Dec 2 2020, 9:13 AM · Restricted Project
Dec 2 2020, 9:12 AM · Restricted Project
Nov 7 2020
Nov 7 2020
Use backticks in ReleaseNotes
All changes introduced by this are now done in https://reviews.llvm.org/D90972
Flow updated the summary of D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy.
In D90972#2380640, @sylvestre.ledru wrote:By the way, while you are working on it, why not removing the python extension when installing? We don't need to know that it is Python code.
Flow added reviewers for D91001: [clang-tidy] Install run-clang-tidy.py as run-clang-tidy: alexfh, sylvestre.ledru, Eugene.Zelenko.
Flow added reviewers for D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy: alexfh, sylvestre.ledru, Eugene.Zelenko.
Adding involved persons from https://reviews.llvm.org/D12700 as reviewers, which was the change introducing share/clang/ as install location.
Nov 6 2020
Nov 6 2020
May 20 2020
May 20 2020
Flow added a comment to D79575: Extend Compilation Database by an optional field for compiler arguments.
In D79575#2026887, @sammccall wrote:Hi, thanks for sending this! I'm sympathetic to wanting this fixed but a bit skeptical about changing it :-) It would be good to discuss the tradeoffs, and in any case document whether we expect this problem to be solved by build systems, consuming tools, or in the format.
Flow updated the diff for D79575: Extend Compilation Database by an optional field for compiler arguments.
Instead of repeating the complete string of compiler arguments in the new optional field, simply have them provide an index into the respective 'command' or 'arguments' field values.
May 7 2020
May 7 2020
Flow added a reviewer for D79575: Extend Compilation Database by an optional field for compiler arguments: silvas.