https://github.com/philnik777 approved this pull request.
LGTM with the libunwind changes removed.
https://github.com/llvm/llvm-project/pull/86658
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
philnik777 wrote:
Yes, it would be the only buildkite one (AFAIK). I don't think that would make
much of a difference though.
https://github.com/llvm/llvm-project/pull/86658
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
emaste wrote:
> The C++26 job is part of the first jobs that get run'
Oops, I did miss that. I think this would be the only buildkite job using
`generic-cxx26`
https://github.com/llvm/llvm-project/pull/86658
___
cfe-commits mailing list
@@ -39,6 +39,9 @@
# if defined(__HAIKU__)
# define _LIBUNWIND_TARGET_HAIKU 1
# endif
+#if defined(__FreeBSD__)
emaste wrote:
Yes, libunwind change was included by accident - I just let the CI jobs finish
rather than restarting; I will update the pull req to
@@ -39,6 +39,9 @@
# if defined(__HAIKU__)
# define _LIBUNWIND_TARGET_HAIKU 1
# endif
+#if defined(__FreeBSD__)
philnik777 wrote:
Are these changes related?
https://github.com/llvm/llvm-project/pull/86658
___
philnik777 wrote:
The C++26 job is part of the first jobs that get run. Did you just miss it, or
do you mean something else?
https://github.com/llvm/llvm-project/pull/86658
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
llvmbot wrote:
@llvm/pr-subscribers-libunwind
Author: Ed Maste (emaste)
Changes
As discussed in #86320; opening this PR for CI.
It looks like (prior to this change) there are no C++26 jobs right now.
---
Full diff: https://github.com/llvm/llvm-project/pull/86658.diff
3 Files Affected:
https://github.com/emaste created
https://github.com/llvm/llvm-project/pull/86658
As discussed in #86320; opening this PR for CI.
It looks like (prior to this change) there are no C++26 jobs right now.
>From 6cf1345feee41ec3bad51ba853091e35723c85ab Mon Sep 17 00:00:00 2001
From: Dmitry Chagin