peterwaller-arm wrote:
>So what should we (Amazon Linux) do to help with this ?
What I think is needed is:
* Public consensus on how to detect the triple (or at least the vendor) on
Amazon Linux. This would help coming from you.
* Consensus on how/when to insert the triple into `LLVM_INFERRED
peterwaller-arm wrote:
OK, Thanks for the input all. I'm abandoning this change, I'm not following up
for the time being if someone is able to fix this.
Part of the problem is that it's not clear what a clean/correct cmake check
would be to determine the triple. My current thinking is that cma
https://github.com/peterwaller-arm closed
https://github.com/llvm/llvm-project/pull/109263
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
peterwaller-arm wrote:
Consensus expressed so far seems to be in favour of merging but I'll wait to
give further opportunity for comment or objection before merging on Tuesday 8th
around 0900 UTC.
https://github.com/llvm/llvm-project/pull/109263
___
peterwaller-arm wrote:
@MaskRay please would you kindly reconsider aarch64-amazon-linux triple as an
exception given the presence of x86_64 equivalent?
I've explored that `LLVM_DEFAULT_TARGET_TRIPLE` if unspecified comes from
`LLVM_INFERRED_HOST_TRIPLE` which comes from config.guess in here:
peterwaller-arm wrote:
> The CMake build system could be adjusted to make
> -DLLVM_DEFAULT_TARGET_TRIPLE=aarch64-amazon-linux implied on Amazon Linux,
> though.
Thanks - Is there precedence for this? Do you have any ideas about how to
detect the correct default triple?
https://github.com/llv
peterwaller-arm wrote:
I've checked the buildkite failures are in cfi-standalone-lld-thinlto-x86_64,
which I believe are unrelated to this change.
https://github.com/llvm/llvm-project/pull/109263
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/peterwaller-arm ready_for_review
https://github.com/llvm/llvm-project/pull/109263
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/peterwaller-arm created
https://github.com/llvm/llvm-project/pull/109263
I note that the comment nearby says "They are not needed when the user has
correct LLVM_DEFAULT_TARGET_TRIPLE" and "The lists should shrink over time".
The behviour was diverged between X86 and aarch64:
peterwaller-arm wrote:
> @peterwaller-arm I noticed that in
> `llvm/test/CodeGen/AArch64/combine_andor_with_cmps.ll`, `FMAXNUM_IEEE` is
> claimed that it is not supported. While I noticed that `fmaxnm` follows the
> rules of `maxNUM` of IEEE754-2008. Is there any other limitation of `fmaxnm`?
https://github.com/peterwaller-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/90734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/peterwaller-arm approved this pull request.
LGTM, thanks for supplying a quick fix and iterating this, and thanks for
waiting for input on the RFC.
https://github.com/llvm/llvm-project/pull/89638
___
cfe-commits mailing list
cfe-com
https://github.com/peterwaller-arm edited
https://github.com/llvm/llvm-project/pull/89638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -217,7 +217,7 @@ function test-armv7m-picolibc() {
"${@}"
${NINJA} -vC "${BUILD_DIR}/compiler-rt" install
-mv "${BUILD_DIR}/install/lib/armv7m-none-unknown-eabi"/*
"${BUILD_DIR}/install/lib"
+mv "${BUILD_DIR}/install/lib/armv7m-unknown-none-eabi"/*
"${
https://github.com/peterwaller-arm requested changes to this pull request.
Thanks for doing this and waiting for RFC comments.
I think this can be merged, the one thing which needs fixing is removing the
change I've commented on so that the picolibc builder can succeed (it uses a
version of cl
peterwaller-arm wrote:
Thanks @wzssyqa again for implementing this and splitting bits out per Sander's
suggestion. I propose we merge these changes tomorrow, on the understanding
that each of the two patches (this and #90313) are passing tests. It looks to
me like you're working towards that g
peterwaller-arm wrote:
I've just posted an RFC to discourse about how to handle triple normalization:
https://discourse.llvm.org/t/rfc-baremetal-target-triple-normalization/78524
https://github.com/llvm/llvm-project/pull/89638
___
cfe-commits mailing
peterwaller-arm wrote:
I have tested building a toolchain using `aarch64-none-elf`, and the problems
described in my comment on
https://github.com/llvm/llvm-project/pull/89234#issuecomment-2069019606 are
resolved. The only change I had to make was to update some references to
`aarch64-none-el
https://github.com/peterwaller-arm closed
https://github.com/llvm/llvm-project/pull/79454
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/peterwaller-arm updated
https://github.com/llvm/llvm-project/pull/79454
>From 18fbfe3d757372fc904146a9156e8af466c9f013 Mon Sep 17 00:00:00 2001
From: Peter Waller
Date: Thu, 25 Jan 2024 14:29:45 +
Subject: [PATCH] [AArch64][compiler-rt] Avoid use of libc header in
sme-li
https://github.com/peterwaller-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/70970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
peterwaller-arm wrote:
Worth a simultaneous update to the langref?
https://github.com/llvm/llvm-project/blob/e2564b27472638d2e2019e6cd2fc6d6d608f8b8c/llvm/docs/LangRef.rst#L5080-L5106
https://github.com/llvm/llvm-project/pull/70970
___
cfe-commits mai
Author: Peter Waller
Date: 2022-11-03T07:55:57Z
New Revision: 8cb9e3c3ce1e7e1658921f90420b68ca16bb98fc
URL:
https://github.com/llvm/llvm-project/commit/8cb9e3c3ce1e7e1658921f90420b68ca16bb98fc
DIFF:
https://github.com/llvm/llvm-project/commit/8cb9e3c3ce1e7e1658921f90420b68ca16bb98fc.diff
LOG:
Author: Peter Waller
Date: 2022-07-07T10:55:40Z
New Revision: 2db2a4e112406e97bad82b8e78492a472c13588e
URL:
https://github.com/llvm/llvm-project/commit/2db2a4e112406e97bad82b8e78492a472c13588e
DIFF:
https://github.com/llvm/llvm-project/commit/2db2a4e112406e97bad82b8e78492a472c13588e.diff
LOG:
Author: Peter Waller
Date: 2021-05-20T16:22:50Z
New Revision: 2d574a110440597eefe1b2a8b6144e4e89c21d05
URL:
https://github.com/llvm/llvm-project/commit/2d574a110440597eefe1b2a8b6144e4e89c21d05
DIFF:
https://github.com/llvm/llvm-project/commit/2d574a110440597eefe1b2a8b6144e4e89c21d05.diff
LOG:
Author: Peter Waller
Date: 2020-12-10T12:43:14Z
New Revision: 2315e9874c92bf625ec84a5f45a4fa28bfbc16ce
URL:
https://github.com/llvm/llvm-project/commit/2315e9874c92bf625ec84a5f45a4fa28bfbc16ce
DIFF:
https://github.com/llvm/llvm-project/commit/2315e9874c92bf625ec84a5f45a4fa28bfbc16ce.diff
LOG:
Author: Peter Waller
Date: 2019-10-30T17:01:59Z
New Revision: c75cd3c7f0f924d53f07a9cce60c362751678e0c
URL:
https://github.com/llvm/llvm-project/commit/c75cd3c7f0f924d53f07a9cce60c362751678e0c
DIFF:
https://github.com/llvm/llvm-project/commit/c75cd3c7f0f924d53f07a9cce60c362751678e0c.diff
LOG:
Author: Peter Waller
Date: 2019-10-30T10:42:22Z
New Revision: 6bf55804924d5a1d902925ad080b1a2b57c5c75c
URL:
https://github.com/llvm/llvm-project/commit/6bf55804924d5a1d902925ad080b1a2b57c5c75c
DIFF:
https://github.com/llvm/llvm-project/commit/6bf55804924d5a1d902925ad080b1a2b57c5c75c.diff
LOG:
28 matches
Mail list logo
