llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-arm-ubuntu` running
on `linaro-lldb-arm-ubuntu` while building `clang,llvm` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/18/builds/15309
Here is the relevant piece of the bui
https://github.com/alanzhao1 updated
https://github.com/llvm/llvm-project/pull/137737
>From 1f0529aa5fbf9c4c1b3f0e58eaf5e527dc1605f2 Mon Sep 17 00:00:00 2001
From: Alan Zhao
Date: Mon, 28 Apr 2025 16:21:29 -0700
Subject: [PATCH 1/3] [clang] Implement JSON formatted -ftime-report
This patch add
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `ml-opt-dev-x86-64` running
on `ml-opt-dev-x86-64-b2` while building `clang,llvm` at step 4
"cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/137/builds/17770
Here is the relevant pie
https://github.com/alanzhao1 closed
https://github.com/llvm/llvm-project/pull/137737
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
Changes LGTM, but they should come with a release note in
`clang/docs/ReleaseNotes.rst` so users know about the improvement, and we
should also start documenting more about time reports (orthogonal to this PR).
https://github.com/llvm
@@ -296,7 +296,14 @@ int cc1_main(ArrayRef Argv, const char
*Argv0, void *MainAddr) {
// If any timers were active but haven't been destroyed yet, print their
// results now. This happens in -disable-free mode.
- llvm::TimerGroup::printAll(llvm::errs());
+ // llvm::Tim
https://github.com/alanzhao1 updated
https://github.com/llvm/llvm-project/pull/137737
>From 1f0529aa5fbf9c4c1b3f0e58eaf5e527dc1605f2 Mon Sep 17 00:00:00 2001
From: Alan Zhao
Date: Mon, 28 Apr 2025 16:21:29 -0700
Subject: [PATCH 1/2] [clang] Implement JSON formatted -ftime-report
This patch add
@@ -442,10 +442,6 @@ void TimerGroup::clearAll() {
void TimerGroup::printJSONValue(raw_ostream &OS, const PrintRecord &R,
const char *suffix, double Value) {
- assert(yaml::needsQuotes(Name) == yaml::QuotingType::None &&
aeuban
https://github.com/aeubanks edited
https://github.com/llvm/llvm-project/pull/137737
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/aeubanks approved this pull request.
lgtm, although I'd like @AaronBallman to take a quick look to make sure this
seems ok (e.g. naming)
https://github.com/llvm/llvm-project/pull/137737
___
cfe-commits mailing list
cfe-commits@lists
alanzhao1 wrote:
The buildkite failure can be ignored; the offending test
(Unwind/split-machine-functions.test) also fails on `main` (or rather the
parent commit of this PR's branch).
https://github.com/llvm/llvm-project/pull/137737
___
cfe-commits m
alanzhao1 wrote:
@aeubanks this PR doesn't implement redirecting the output to a file, but I
figured I'd implement that in another PR to keep things organized.
https://github.com/llvm/llvm-project/pull/137737
___
cfe-commits mailing list
cfe-commits@l
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Alan Zhao (alanzhao1)
Changes
This patch adds a new flag, -ftime-report-json, which outputs the same
information as -ftime-report but as JSON instead of -ftime-report's pretty
printed format.
---
Full diff: https://github.com/llvm/llvm-p
https://github.com/alanzhao1 created
https://github.com/llvm/llvm-project/pull/137737
This patch adds a new flag, -ftime-report-json, which outputs the same
information as -ftime-report but as JSON instead of -ftime-report's pretty
printed format.
>From 1f0529aa5fbf9c4c1b3f0e58eaf5e527dc1605f
14 matches
Mail list logo
