Re: [PING] [PATCH] testsuite: Disable finite math only for test [PR115826]

2024-07-23 Thread Torbjorn SVENSSON
On 2024-07-23 09:59, Richard Biener wrote: On Tue, 23 Jul 2024, Torbjorn SVENSSON wrote: Gentle ping :) As mentioned in the ticket, I would like to target this to trunk and releases/gcc-14. OK. Richard. Pushed as releases/gcc-14.1.0-331-ga544898f6dd and basepoints/gcc-15-2224

[PING] [PATCH] testsuite: Disable finite math only for test [PR115826]

2024-07-23 Thread Torbjorn SVENSSON
Gentle ping :) As mentioned in the ticket, I would like to target this to trunk and releases/gcc-14. Kind regards, Torbjörn On 2024-07-15 12:16, Torbjörn SVENSSON wrote: As the test case requires +-Inf and NaN to work and -ffast-math is added by default for arm-none-eabi, re-enable

Re: [PATCH] testsuite: Allow vst1 instruction

2024-07-22 Thread Torbjorn SVENSSON
On 2024-07-19 17:22, Richard Earnshaw (lists) wrote: On 19/07/2024 16:10, Torbjorn SVENSSON wrote: On 2024-07-19 16:36, Richard Earnshaw (lists) wrote: On 19/07/2024 14:07, Torbjörn SVENSSON wrote: Ok for trunk and releases/gcc-14? -- On Cortex-A7 with -mfloat-abi=hard and -mfpu=neon

Re: [PATCH] testsuite: Allow vst1 instruction

2024-07-19 Thread Torbjorn SVENSSON
On 2024-07-19 16:36, Richard Earnshaw (lists) wrote: On 19/07/2024 14:07, Torbjörn SVENSSON wrote: Ok for trunk and releases/gcc-14? -- On Cortex-A7 with -mfloat-abi=hard and -mfpu=neon, the following instructions are generated for the test case: vldrd16, .L3 vst1.32

Re: [PATCH] testsuite: Align testcase with implementation [PR105090]

2024-07-10 Thread Torbjorn SVENSSON
On 2024-07-10 18:41, Richard Earnshaw (lists) wrote: On 10/07/2024 17:26, Torbjörn SVENSSON wrote: Is this ok for the following branches? - trunk - releases/gcc-14 - releases/gcc-13 -- Since r13-1006-g2005b9b888eeac, the test case copysign_softfloat_1.c no longer contains any lsr

Re: [PATCH v3 1/2] arm: Zero/Sign extends for CMSE security on Armv8-M.baseline [PR115253]

2024-06-12 Thread Torbjorn SVENSSON
On 2024-06-11 15:59, Richard Earnshaw (lists) wrote: On 10/06/2024 15:04, Torbjörn SVENSSON wrote: Properly handle zero and sign extension for Armv8-M.baseline as Cortex-M23 can have the security extension active. Currently, there is an internal compiler error on Cortex-M23 for the epilog

Re: [PATCH v3 2/2] testsuite: Fix expand-return CMSE test for Armv8.1-M [PR115253]

2024-06-12 Thread Torbjorn SVENSSON
On 2024-06-11 16:00, Richard Earnshaw (lists) wrote: On 10/06/2024 15:04, Torbjörn SVENSSON wrote: For Armv8.1-M, the clearing of the registers is handled differently than for Armv8-M, so update the test case accordingly. gcc/testsuite/ChangeLog: PR target/115253 *

Re: [PATCH v2 1/2] arm: Zero/Sign extends for CMSE security on Armv8-M.baseline [PR115253]

2024-06-10 Thread Torbjorn SVENSSON
Hi Andre, Thanks for the review! Please see my questions below. On 2024-06-10 12:37, Andre Vieira (lists) wrote: Hi Torbjorn, Thanks for this, I have some comments below. On 07/06/2024 09:56, Torbjörn SVENSSON wrote: Properly handle zero and sign extension for Armv8-M.baseline as Cortex-M23

Re: [PATCH v2] testsuite: Verify r0-r3 are extended with CMSE

2024-05-22 Thread Torbjorn SVENSSON
: d9c89402b54be4c15bb3c7bcce3465f534746204 releases/gcc-11: 08ca81e4b49bda153d678a372df7f7143a94f4ad Kind regards, Torbjörn On 2024-05-22 13:54, Richard Earnshaw (lists) wrote: On 22/05/2024 12:14, Torbjorn SVENSSON wrote: Hello Richard, Thanks for the reply. From my point of view, at least the -fshort-enums

Re: [PATCH v2] testsuite: Verify r0-r3 are extended with CMSE

2024-05-22 Thread Torbjorn SVENSSON
regards, Torbjörn On 2024-05-22 12:55, Richard Earnshaw (lists) wrote: On 06/05/2024 12:50, Torbjorn SVENSSON wrote: Hi, Forgot to mention when I sent the patch that I would like to commit it to the following branches: - releases/gcc-11 - releases/gcc-12 - releases/gcc-13 - releases/gcc-14 - trunk

[PING^2] [PATCH v2] testsuite: Verify r0-r3 are extended with CMSE

2024-05-21 Thread Torbjorn SVENSSON
Gentle ping! Kind regards, Torbjörn On 2024-05-14 13:01, Torbjorn SVENSSON wrote: Hi, I'm not sure if the previous "ok" from Richard on the v1 is enough for this or if there needs another approval. Adding extra maintainers since Richard Earnshaw appears to be busy the past wee

[PING] [PATCH v2] testsuite: Verify r0-r3 are extended with CMSE

2024-05-14 Thread Torbjorn SVENSSON
Hi, I'm not sure if the previous "ok" from Richard on the v1 is enough for this or if there needs another approval. Adding extra maintainers since Richard Earnshaw appears to be busy the past weeks. Kind regards, Torbjörn On 2024-05-06 13:50, Torbjorn SVENSSON wrote: H

Re: [PATCH v2] testsuite: Verify r0-r3 are extended with CMSE

2024-05-06 Thread Torbjorn SVENSSON
Hi, Forgot to mention when I sent the patch that I would like to commit it to the following branches: - releases/gcc-11 - releases/gcc-12 - releases/gcc-13 - releases/gcc-14 - trunk Kind regards, Torbjörn On 2024-05-02 12:50, Torbjörn SVENSSON wrote: Add regression test to the existing

Re: [PATCH] testsuite: Verify r0-r3 are extended with CMSE

2024-04-30 Thread Torbjorn SVENSSON
On 2024-04-30 17:11, Richard Earnshaw (lists) wrote: On 27/04/2024 15:13, Torbjörn SVENSSON wrote: Add regression test to the existing zero/sign extend tests for CMSE to verify that r0, r1, r2 and r3 are properly extended, not just r0. Test is done using -O0 to ensure the instructions are

Re: [PATCH] arm: Zero/Sign extends for CMSE security

2024-04-26 Thread Torbjorn SVENSSON
Hi, On 2024-04-25 16:25, Richard Ball wrote: Hi Torbjorn, Thanks very much for the comments. I think given that the code that handles this, is within a FOREACH_FUNCTION_ARGS loop. It seems a fairly safe assumption that if the code works for one that it will work for all. To go back and add

Re: [PATCH] arm: Zero/Sign extends for CMSE security

2024-04-25 Thread Torbjorn SVENSSON
Hi, On 2024-04-24 17:55, Richard Ball wrote: This patch makes the following changes: 1) When calling a secure function from non-secure code then any arguments smaller than 32-bits that are passed in registers are zero- or sign-extended. 2) After a non-secure function returns into secure

Re: [PING^5] Re: [PATCH] analyzer: deal with -fshort-enums

2024-04-10 Thread Torbjorn SVENSSON
Ping! Kind regards, Torbjörn On 2024-03-25 15:59, Yvan ROUX - foss wrote: Ping! Rgds, Yvan From: Torbjorn SVENSSON - foss Sent: Friday, March 15, 2024 11:32 AM To: David Malcolm; Alexandre Oliva Cc: gcc-patches@gcc.gnu.org; Yvan ROUX - foss Subject

Re: [PATCH] testsuite: Define _POSIX_C_SOURCE for test

2024-03-18 Thread Torbjorn SVENSSON
On 2024-03-17 18:48, Mike Stump wrote: On Mar 10, 2024, at 10:26 AM, Torbjörn SVENSSON wrote: Ok for trunk? Ok. Pushed as basepoints/gcc-14-9513-g58753dba800 to trunk. Kind regards, Torbjörn

[PING] Re: [PATCH] testsuite: Define _POSIX_C_SOURCE for test

2024-03-16 Thread Torbjorn SVENSSON
Ping! Kind regards, Torbjörn On 2024-03-10 18:26, Torbjörn SVENSSON wrote: Ok for trunk? -- As the tests assume that strndup() is visible (only part of POSIX.1-2008) define the guard to ensure that it's visible. Currently, glibc appears to always have this defined in C++, newlib does not.

[PING^3] Re: [PATCH] analyzer: deal with -fshort-enums

2024-03-15 Thread Torbjorn SVENSSON
Ping! Kind regards, Torbjörn On 2024-03-08 10:14, Torbjorn SVENSSON wrote: Ping! Kind regards, Torbjörn On 2024-02-22 09:51, Torbjorn SVENSSON wrote: Ping! Kind regards, Torbjörn On 2024-02-07 17:21, Torbjorn SVENSSON wrote: Hi, Is it okay to backport

Re: [PATCH v2] testsuite: xfail test for short_enums

2024-03-13 Thread Torbjorn SVENSSON
On 2024-03-12 14:21, Jason Merrill wrote: On 3/11/24 06:23, Torbjörn SVENSSON wrote: Changes compared to v1: - Added reference to r14-6517-gb7e4a4c626e in dg-bogus comment - Changed arm-*-* to short_enums in target selector - Updated commit message to align with above changes As the entire

[PING^2] Re: [PATCH] analyzer: deal with -fshort-enums

2024-03-08 Thread Torbjorn SVENSSON
Ping! Kind regards, Torbjörn On 2024-02-22 09:51, Torbjorn SVENSSON wrote: Ping! Kind regards, Torbjörn On 2024-02-07 17:21, Torbjorn SVENSSON wrote: Hi, Is it okay to backport 3cbab07b08d2f3a3ed34b6ec12e67727c59d285c to releases/gcc-13? Without this backport, I see these failures

Re: [PATCH] arm: Fixed C23 call compatibility with arm-none-eabi

2024-03-04 Thread Torbjorn SVENSSON
On 2024-03-01 15:58, Richard Earnshaw (lists) wrote: On 19/02/2024 09:13, Torbjörn SVENSSON wrote: Ok for trunk and releases/gcc-13? Regtested on top of 945cb8490cb for arm-none-eabi, without any regression. Backporting to releases/gcc-13 will change -std=c23 to -std=c2x. Jakub has just

[PING] Re: [PATCH 1/2] c-family: -Waddress-of-packed-member and casts

2024-02-22 Thread Torbjorn SVENSSON
Ping! Kind regards, Torbjörn On 2024-02-07 17:19, Torbjorn SVENSSON wrote: Hi, Is it okay to backport b7e4a4c626eeeb32c291d5bbbaa148c5081b6bfd to releases/gcc-13? Without this backport, I see these failures on arm-none-eabi: FAIL: gcc.dg/analyzer/null-deref-pr108251

[PING] Re: [PATCH] analyzer: deal with -fshort-enums

2024-02-22 Thread Torbjorn SVENSSON
Ping! Kind regards, Torbjörn On 2024-02-07 17:21, Torbjorn SVENSSON wrote: Hi, Is it okay to backport 3cbab07b08d2f3a3ed34b6ec12e67727c59d285c to releases/gcc-13? Without this backport, I see these failures on arm-none-eabi: FAIL: gcc.dg/analyzer/switch-enum-1.c  (test for bogus messages

Re: [PATCH] testsuite: Define _POSIX_SOURCE for tests [PR113278]

2024-02-15 Thread Torbjorn SVENSSON
On 2024-02-15 18:18, Mike Stump wrote: On Feb 15, 2024, at 9:03 AM, Torbjörn SVENSSON wrote: Ok for trunk? Ok. Pushed as 8e8c2d2b34971bb29e74341a3efc625f1db06639. gcc/testsuite/ChangeLog: PR113278 * c-c++-common/analyzer/fileno-1.c: Define _POSIX_SOURCE. *

Re: [PATCH] testsuite: Update test case to comply with GCC14 changes

2024-02-11 Thread Torbjorn SVENSSON
On 2024-02-11 20:01, Mike Stump wrote: On Feb 10, 2024, at 7:21 AM, Torbjörn SVENSSON wrote: I have confirmed that this updated pr97969.c file still hangs with gcc-arm-none-eabi-9-2020-q2-update as mentioned in comment 2 of PR97969. Ok for trunk? Ok. Pushed as

Re: [PATCH] call maybe_return_this in build_clone

2024-02-09 Thread Torbjorn SVENSSON
On 2024-02-09 19:57, Jason Merrill wrote: On 2/7/24 11:24, Torbjorn SVENSSON wrote: Hi, Is it okay to backport 0d24289d129639efdc79338a64188d6d404375e8 to releases/gcc-13? OK. Pushed as 583bd84075d23cde5fd489ab726093060870d773.

Re: [PATCH] [testsuite] tsvc: skip include malloc.h when unavailable

2024-02-09 Thread Torbjorn SVENSSON
On 2024-02-09 11:34, Richard Biener wrote: On Fri, Feb 9, 2024 at 11:33 AM Torbjorn SVENSSON wrote: Hi, Is it okay to backport 2f20d6296087cae51f55eeecb3efefe786191fd6 to releases/gcc-13? Yes. Pushed as 5b3dcff46780192a2e526bc434d61c8626898050.

Re: [PATCH] [testsuite] tsvc: skip include malloc.h when unavailable

2024-02-09 Thread Torbjorn SVENSSON
Hi, Is it okay to backport 2f20d6296087cae51f55eeecb3efefe786191fd6 to releases/gcc-13? Without this backport, I see about 150 failures on arm-none-eabi, an example of them is: FAIL: gcc.dg/vect/tsvc/vect-tsvc-s000.c (test for excess errors) Kind regards, Torbjörn On 2023-05-24 11:02,

Re: [PATCH v2] libstdc++: optimize bit iterators assuming normalization [PR110807]

2024-02-09 Thread Torbjorn SVENSSON
On 2024-02-07 17:36, Jonathan Wakely wrote: On Wed, 7 Feb 2024 at 16:25, Torbjorn SVENSSON mailto:torbjorn.svens...@foss.st.com>> wrote: Hi, Is it okay to backport e39b3e02c27bd771a07e385f9672ecf1a45ced77 to releases/gcc-13? It would als

Re: [PATCH v2] testsuite: Pattern does not match when using --specs=nano.specs

2024-02-09 Thread Torbjorn SVENSSON
On 2024-02-09 01:07, Mike Stump wrote: On Feb 8, 2024, at 9:44 AM, Torbjörn SVENSSON wrote: Changes since v1: - Replaced .* with [^\r\n]* to avoid matching newline. Ok for trunk and releases/gcc-13? Ok. Pushed as 1175d1b35ce7bf8ee7c9b37b334370f01eb95335 and

Re: [PATCH] c++: for contracts, cdtors never return this

2024-02-09 Thread Torbjorn SVENSSON
On 2024-02-07 20:46, Jason Merrill wrote: On 2/7/24 11:22, Torbjorn SVENSSON wrote: Hi, Is it okay to backport 71804526d3a71a8c0f189a89ce3aa615784bfd8b to releases/gcc-13? OK. Pushed as eae51472f68d9f922aa3a1a636f81467bfdae87a.

Re: Ping: [PATCH v2 1/2] testsuite: Add dg-require-atomic-cmpxchg-word

2024-02-07 Thread Torbjorn SVENSSON
On 2024-02-07 17:33, Jonathan Wakely wrote: On Wed, 7 Feb 2024 at 16:31, Torbjorn SVENSSON mailto:torbjorn.svens...@foss.st.com>> wrote: Hi, Is it okay to backport 62b29347c38394ae32858f2301aa9aa65205984e, 2a4d9e4f533c77870cc0eb60fbbd8047da

Re: [PATCH 1/2] Fix contracts-tmpl-spec2.C on targets where plain char is unsigned by default

2024-02-07 Thread Torbjorn SVENSSON
On 2024-02-07 18:13, Andrew Pinski (QUIC) wrote: -Original Message- From: Torbjorn SVENSSON Sent: Wednesday, February 7, 2024 8:23 AM To: Andrew Pinski (QUIC) ; gcc- patc...@gcc.gnu.org Cc: Yvan Roux Subject: Re: [PATCH 1/2] Fix contracts-tmpl-spec2.C on targets where plain char

Re: Ping: [PATCH v2 1/2] testsuite: Add dg-require-atomic-cmpxchg-word

2024-02-07 Thread Torbjorn SVENSSON
Hi, Is it okay to backport 62b29347c38394ae32858f2301aa9aa65205984e, 2a4d9e4f533c77870cc0eb60fbbd8047da4c7386 and ba0cde8ba2d93b7193050eb5ef3cc6f7a2cdfe61 to releases/gcc-13? Without this backport, I see these failures on arm-none-eabi: FAIL: 29_atomics/atomic/compare_exchange_padding.cc

Re: [PATCH v2] libstdc++: optimize bit iterators assuming normalization [PR110807]

2024-02-07 Thread Torbjorn SVENSSON
Hi, Is it okay to backport e39b3e02c27bd771a07e385f9672ecf1a45ced77 to releases/gcc-13? Without this backport, I see this failure on arm-none-eabi: FAIL: 23_containers/vector/bool/110807.cc (test for excess errors) Kind regards, Torbjörn On 2023-11-09 02:22, Jonathan Wakely wrote: On

Re: [PATCH] call maybe_return_this in build_clone

2024-02-07 Thread Torbjorn SVENSSON
Hi, Is it okay to backport 0d24289d129639efdc79338a64188d6d404375e8 to releases/gcc-13? Without this backport, I see these failures on arm-none-eabi: FAIL: g++.dg/warn/Wuse-after-free3.C -std=gnu++14 (test for excess errors) FAIL: g++.dg/warn/Wuse-after-free3.C -std=gnu++17 (test for

Re: [PATCH 1/2] Fix contracts-tmpl-spec2.C on targets where plain char is unsigned by default

2024-02-07 Thread Torbjorn SVENSSON
Hi, Is it okay to backport 6e15e4e1abed02443a27a69455f4bfa49457c99e to releases/gcc-13? Without this backport, I see this failure on arm-none-eabi: FAIL: g++.dg/contracts/contracts-tmpl-spec2.C output pattern test Kind regards, Torbjörn On 2023-11-26 03:57, Andrew Pinski wrote: Since

Re: [PATCH] c++: for contracts, cdtors never return this

2024-02-07 Thread Torbjorn SVENSSON
Hi, Is it okay to backport 71804526d3a71a8c0f189a89ce3aa615784bfd8b to releases/gcc-13? Without this backport, I see these failures on arm-none-eabi: FAIL: g++.dg/contracts/contracts-ctor-dtor2.C(test for errors, line 23) FAIL: g++.dg/contracts/contracts-ctor-dtor2.C(test for errors,

Re: [PATCH] analyzer: deal with -fshort-enums

2024-02-07 Thread Torbjorn SVENSSON
Hi, Is it okay to backport 3cbab07b08d2f3a3ed34b6ec12e67727c59d285c to releases/gcc-13? Without this backport, I see these failures on arm-none-eabi: FAIL: gcc.dg/analyzer/switch-enum-1.c (test for bogus messages, line 26) FAIL: gcc.dg/analyzer/switch-enum-1.c (test for bogus messages,

Re: [PATCH 1/2] c-family: -Waddress-of-packed-member and casts

2024-02-07 Thread Torbjorn SVENSSON
Hi, Is it okay to backport b7e4a4c626eeeb32c291d5bbbaa148c5081b6bfd to releases/gcc-13? Without this backport, I see these failures on arm-none-eabi: FAIL: gcc.dg/analyzer/null-deref-pr108251-smp_fetch_ssl_fc_has_early-O2.c (test for excess errors) FAIL:

Re: [PATCH] testsuite: Skip intrinsics test if arm

2023-01-15 Thread Torbjorn SVENSSON via Gcc-patches
On 2023-01-12 16:03, Richard Earnshaw wrote: On 19/09/2022 17:16, Torbjörn SVENSSON via Gcc-patches wrote: In the test case, it's clearly written that intrinsics is not implemented on arm*. A simple xfail does not help since there are link error and that would cause an UNRESOLVED testcase

Re: PING [PATCH v3] c++: Allow module name to be a single letter on Windows

2022-11-28 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-11-28 12:21, Nathan Sidwell wrote: On 11/25/22 14:03, Torbjorn SVENSSON wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-November/606528.html ok, thanks! Pushed. Kind regards, Torbjörn On 2022-11-17 14:20, Torbjörn SVENSSON wrote: v1 -> v2: Paths without

PING [PATCH v3] c++: Allow module name to be a single letter on Windows

2022-11-25 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-November/606528.html Kind regards, Torbjörn On 2022-11-17 14:20, Torbjörn SVENSSON wrote: v1 -> v2: Paths without "C:" part can still be absolute if they start with / or \ on Windows. v2 -> v3: Use alternative approach by having

Re: PING^2 [PATCH] testsuite: Windows paths use \ and not /

2022-11-21 Thread Torbjorn SVENSSON via Gcc-patches
Pushed after off-list approval from Jeff Law. On 2022-11-17 17:44, Torbjorn SVENSSON via Gcc-patches wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-November/604896.html Ok for trunk? Kind regards, Torbjörn On 2022-11-02 19:16, Torbjorn SVENSSON wrote: Hi, Ping, https

Re: PING^1 [PATCH] cpp/remap: Only override if string matched

2022-11-21 Thread Torbjorn SVENSSON via Gcc-patches
Hi, On 2022-11-20 21:31, Jeff Law wrote: On 11/2/22 12:21, Torbjorn SVENSSON via Gcc-patches wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/604062.html Ok for trunk? OK.  Sorry for the delay. Thanks for the review. Pushed. Kind regards, Torbjörn jeff

Re: PING^5 [PATCH] testsuite: Verify that module-mapper is available

2022-11-18 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-11-18 09:14, Richard Biener wrote: On Thu, Nov 17, 2022 at 6:09 PM Torbjorn SVENSSON via Gcc-patches wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-November/604895.html Ok for trunk? OK. Pushed. Kind regards, Torbjörn On 2022-11-02 19:13, Torbjorn

PING^5 [PATCH] testsuite: Verify that module-mapper is available

2022-11-17 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-November/604895.html Ok for trunk? Kind regards, Torbjörn On 2022-11-02 19:13, Torbjorn SVENSSON wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/602844.html Ok for trunk? Kind regards, Torbjörn On 2022-10-25

PING^2 [PATCH] cpp/remap: Only override if string matched

2022-11-17 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-November/604898.html Ok for trunk? Kind regards, Torbjörn On 2022-11-02 19:21, Torbjorn SVENSSON wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/604062.html Ok for trunk? Kind regards, Torbjörn On 2022-10-20

PING^2 [PATCH] testsuite: Windows paths use \ and not /

2022-11-17 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-November/604896.html Ok for trunk? Kind regards, Torbjörn On 2022-11-02 19:16, Torbjorn SVENSSON wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/604312.html Ok for trunk? Kind regards, Torbjörn On 2022-10-25

Re: [PATCH] testsuite: Fix mistransformed gcov

2022-11-15 Thread Torbjorn SVENSSON via Gcc-patches
Hi, On 2022-11-16 03:11, Hans-Peter Nilsson wrote: How was r13-2619-g34b9a03353d3fd "gcov: Respect triplet when looking for gcov" tested? I'm having a hard time believing it was tested with a *cross-compiler* *in-build-tree*. I think it was only tested for the special-case of an installed

Re: [PATCH] c++: Allow module name to be a single letter on Windows

2022-11-08 Thread Torbjorn SVENSSON via Gcc-patches
Hi Nathan, On 2022-11-08 00:03, Nathan Sidwell wrote: On 11/3/22 11:06, Torbjorn SVENSSON wrote: On 2022-11-03 15:17, Nathan Sidwell wrote: On 10/28/22 05:15, Torbjörn SVENSSON wrote: On Windows, the ':' character is special and when the module name is a single character, like

Re: PING^1 [PATCH] arm: Allow to override location of .gnu.sgstubs section

2022-11-04 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-11-03 10:44, Kyrylo Tkachov wrote: -Original Message- From: Torbjorn SVENSSON Sent: Wednesday, November 2, 2022 6:19 PM To: gcc-patches@gcc.gnu.org Cc: Kyrylo Tkachov Subject: PING^1 [PATCH] arm: Allow to override location of .gnu.sgstubs section Hi, Ping, https

Re: [PATCH v2] c++: Use in-process client when networking is disabled

2022-11-04 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-11-03 15:09, Nathan Sidwell wrote: On 11/3/22 09:48, Torbjorn SVENSSON wrote: Hello Nathan, On 2022-11-03 14:13, Nathan Sidwell wrote: On 11/3/22 05:37, Torbjörn SVENSSON wrote: v1 -> v2: Updated expression in bad-mapper-3.C Ok for trunk? --- Without the patch, the out

Re: [PATCH] c++: Allow module name to be a single letter on Windows

2022-11-03 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-11-03 15:17, Nathan Sidwell wrote: On 10/28/22 05:15, Torbjörn SVENSSON wrote: On Windows, the ':' character is special and when the module name is a single character, like 'A', then the flatname would be (for example) 'A:Foo'. On Windows, 'A:Foo' is treated as an absolute path by

Re: [PATCH v2] c++: Use in-process client when networking is disabled

2022-11-03 Thread Torbjorn SVENSSON via Gcc-patches
Hello Nathan, On 2022-11-03 14:13, Nathan Sidwell wrote: On 11/3/22 05:37, Torbjörn SVENSSON wrote: v1 -> v2: Updated expression in bad-mapper-3.C Ok for trunk? --- Without the patch, the output for bad-mapper-3.C would be: /src/gcc/gcc/testsuite/g++.dg/modules/bad-mapper-3.C:2:1: error:

PING^1 [PATCH] cpp/remap: Only override if string matched

2022-11-02 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/604062.html Ok for trunk? Kind regards, Torbjörn On 2022-10-20 22:48, Torbjörn SVENSSON wrote: For systems with HAVE_DOS_BASED_FILE_SYSTEM set, only override the pointer if the backslash pattern matches. Output without this

PING^1 [PATCH] arm: Allow to override location of .gnu.sgstubs section

2022-11-02 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603878.html Kind regards, Torbjörn On 2022-10-19 11:42, Torbjörn SVENSSON wrote: Depending on the DejaGNU board definition, the .gnu.sgstubs section might be placed on different locations in order to suite the target. With this

PING^1 [PATCH] testsuite: Windows paths use \ and not /

2022-11-02 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/604312.html Ok for trunk? Kind regards, Torbjörn On 2022-10-25 17:15, Torbjörn SVENSSON wrote: Without this patch, the following error is reported on Windows: In file included from

Re: PING^4 [PATCH] testsuite: Verify that module-mapper is available

2022-11-02 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/602844.html Ok for trunk? Kind regards, Torbjörn On 2022-10-25 16:24, Torbjorn SVENSSON via Gcc-patches wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603544.html Kind regards, Torbjörn On 2022-10-14

Re: [PATCH] IRA: Make sure array is big enough

2022-10-27 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-10-26 22:26, Vladimir Makarov wrote: On 2022-10-25 06:01, Torbjörn SVENSSON wrote: In commit 081c96621da, the call to resize_reg_info() was moved before the call to remove_scratches() and the latter one can increase the number of regs and that would cause an out of bounds usage on

PING^3 [PATCH] testsuite: Verify that module-mapper is available

2022-10-25 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603544.html Kind regards, Torbjörn On 2022-10-14 09:42, Torbjorn SVENSSON wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/602843.html Kind regards, Torbjörn On 2022-10-05 11:17, Torbjorn SVENSSON wrote

Re: [PATCH] lto: Always quote path to touch

2022-10-24 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-10-24 10:07, Richard Biener wrote: On Fri, 21 Oct 2022, Torbj?rn SVENSSON wrote: When generating the makefile, make sure that the paths are quoted so that a native Windows path works within Cygwin. Without this patch, this error is reported by the DejaGNU test suite: make:

Re: [PATCH v4] testsuite: Sanitize fails for SP FPU on Arm

2022-10-20 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-10-20 20:19, Joseph Myers wrote: On Wed, 19 Oct 2022, Torbjörn SVENSSON via Gcc-patches wrote: This patch stops reporting fails for Arm targets with single precision floating point unit for types wider than 32 bits (the width of float on arm-none-eabi). As reported in PR102017,

Re: [PATCH] arm: Allow to override location of .gnu.sgstubs section

2022-10-19 Thread Torbjorn SVENSSON via Gcc-patches
Hi Christophe, On 2022-10-19 11:52, Christophe Lyon wrote: Hi Torbjörn, This looks like a nice improvement to me ;-) On 10/19/22 11:42, Torbjörn SVENSSON via Gcc-patches wrote: Depending on the DejaGNU board definition, the .gnu.sgstubs section might be placed on different locations in order

PING^1 [PATCH v3] testsuite: Sanitize fails for SP FPU on Arm

2022-10-14 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603054.html Kind regards, Torbjörn On 2022-10-07 15:28, Torbjörn SVENSSON wrote: This patch stops reporting fails for Arm targets with single precision floating point unit for types wider than 32 bits (the width of float on

PING^2 [PATCH] testsuite: Verify that module-mapper is available

2022-10-14 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-October/602843.html Kind regards, Torbjörn On 2022-10-05 11:17, Torbjorn SVENSSON wrote: Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-September/602111.html Kind regards, Torbjörn On 2022-09-23 14:03, Torbjörn SVENSSON

Re: PING^1 [PATCH] testsuite: 'b' instruction can't do long enough jumps

2022-10-05 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-10-05 11:51, Kyrylo Tkachov wrote: Hi Torbjörn, -Original Message- From: Torbjorn SVENSSON Sent: Wednesday, October 5, 2022 10:28 AM To: Kyrylo Tkachov Cc: Yvan Roux Subject: Fwd: PING^1 [PATCH] testsuite: 'b' instruction can't do long enough jumps Hi Kyrill, I checked

Re: [PATCH] testsuite: /dev/null is not accessible on Windows

2022-10-05 Thread Torbjorn SVENSSON via Gcc-patches
Hi, On 2022-10-05 11:34, Jonathan Yong via Gcc-patches wrote: On 9/29/22 17:38, Torbjörn SVENSSON via Gcc-patches wrote: When running the DejaGNU testsuite on a toolchain built for native Windows, the path /dev/null can't be used to open a stream to void. On native Windows, the resource is

PING^1 [PATCH] testsuite: /dev/null is not accessible on Windows

2022-10-05 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-September/602534.html Kind regards, Torbjörn On 2022-09-29 19:38, Torbjörn SVENSSON wrote: When running the DejaGNU testsuite on a toolchain built for native Windows, the path /dev/null can't be used to open a stream to void. On native

PING^1 [PATCH] testsuite: Verify that module-mapper is available

2022-10-05 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-September/602111.html Kind regards, Torbjörn On 2022-09-23 14:03, Torbjörn SVENSSON wrote: For some test cases, it's required that the optional module mapper "g++-mapper-server" is built. As the server is not required, the test cases

PING^1 [PATCH] testsuite: Windows reports errors with CreateProcess

2022-10-05 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping, https://gcc.gnu.org/pipermail/gcc-patches/2022-September/602541.html Kind regards, Torbjörn On 2022-09-29 20:07, Torbjörn SVENSSON wrote: When the mapper can't be executed, Windows report the error like: .../bad-mapper-1.C: error: failed CreateProcess mapper 'this-will-not-work' On

Re: [PATCH] testsuite: Windows paths use \ and not /

2022-10-01 Thread Torbjorn SVENSSON via Gcc-patches
Hi, I'm really sorry for the mess. I did test my patch, but I just looked for the PASS/FAIL for the excess errors and missed that there was an error with the pattern. In the end, the patch that you pushed is much better. Thanks for fixing the issue in my absence. Kind regards, Torbjörn On

Re: PING^1 [PATCH] testsuite: 'b' instruction can't do long enough jumps

2022-09-28 Thread Torbjorn SVENSSON via Gcc-patches
Hi Christophe! On 2022-09-28 13:55, Christophe Lyon wrote: Hi! On 9/28/22 11:17, Torbjorn SVENSSON via Gcc-patches wrote: Hi, Ping: https://gcc.gnu.org/pipermail/gcc-patches/2022-September/601829.html Kind regards, Torbjörn On 2022-09-19 18:30, Torbjörn SVENSSON wrote: After moving

PING^1 [PATCH] testsuite: Do not prefix linker script with "-Wl,"

2022-09-28 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping: https://gcc.gnu.org/pipermail/gcc-patches/2022-September/601831.html Kind regards, Torbjörn On 2022-09-19 18:57, Torbjörn SVENSSON wrote: The linker script should not be prefixed with "-Wl," - it's not an input file and does not interfere with the new dump output filename strategy.

PING^1 [PATCH] testsuite: 'b' instruction can't do long enough jumps

2022-09-28 Thread Torbjorn SVENSSON via Gcc-patches
Hi, Ping: https://gcc.gnu.org/pipermail/gcc-patches/2022-September/601829.html Kind regards, Torbjörn On 2022-09-19 18:30, Torbjörn SVENSSON wrote: After moving the testglue in commit 9d503515cee, the jump to exit and abort is too far for the 'b' instruction on Cortex-M0. As most of the C

Re: [PATCH v2] testsuite: Only run test on target if VMA == LMA

2022-09-23 Thread Torbjorn SVENSSON via Gcc-patches
Hi Richard, Thanks for your review. Comments below. On 2022-09-23 19:34, Richard Sandiford wrote: Torbjörn SVENSSON via Gcc-patches writes: Checking that the triplet matches arm*-*-eabi (or msp430-*-*) is not enough to know if the execution will enter an endless loop, or if it will give a

Re: [PATCH] testsuite: Sanitize fails for SP FPU on Arm

2022-09-23 Thread Torbjorn SVENSSON via Gcc-patches
Hi Joseph, On 2022-09-23 00:42, Joseph Myers wrote: On Thu, 22 Sep 2022, Torbjörn SVENSSON via Gcc-patches wrote: This patch stops reporting fails for Arm targets with single precision floating point unit for types wider than 32 bits (the width of float on arm-none-eabi). As reported in

Re: [PATCH] testsuite: Disable zero-scratch-regs-{7, 9, 11}.c on arm

2022-09-16 Thread Torbjorn SVENSSON via Gcc-patches
Hi all, Appears that this is just a problem for gcc11 (and perhaps gcc12?). Master already has the needed implementation, so the patch below is not needed. Sorry for the buzz. Kind regards, Torbjörn On 2022-09-15 08:54, Torbjörn SVENSSON wrote: -fzero-call-used-regs=all and

Re: [PATCH v2] gcov: Respect triplet when looking for gcov

2022-09-12 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-09-12 10:40, Martin Liška wrote: On 9/12/22 09:06, Torbjorn SVENSSON via Gcc-patches wrote: On 2022-09-11 21:38, Mikael Morin wrote: Le 11/09/2022 à 18:04, Torbjorn SVENSSON a écrit : Can you fix it for me and submit it or do you want me to send a v3? For trivial things like

Re: [PATCH v2] gcov: Respect triplet when looking for gcov

2022-09-12 Thread Torbjorn SVENSSON via Gcc-patches
On 2022-09-11 21:38, Mikael Morin wrote: Le 11/09/2022 à 18:04, Torbjorn SVENSSON a écrit : Can you fix it for me and submit it or do you want me to send a v3? For trivial things like this, there is no need for a v3 (nor was there for a v2). Do you miss a git write account and need

Re: [PATCH v2] gcov: Respect triplet when looking for gcov

2022-09-11 Thread Torbjorn SVENSSON via Gcc-patches
Hi, On 2022-09-11 16:34, Mikael Morin wrote: Hello, diff --git a/gcc/testsuite/gcc.misc-tests/gcov.exp b/gcc/testsuite/gcc.misc-tests/gcov.exp index 82376d90ac2..a55ce234f6e 100644 --- a/gcc/testsuite/gcc.misc-tests/gcov.exp +++ b/gcc/testsuite/gcc.misc-tests/gcov.exp @@ -24,9 +24,9 @@

Re: [PATCH] Added information about inline assembler in stack calculations (.su files)

2018-11-27 Thread Torbjorn SVENSSON
taget-tree). Torbjörn On 27/11/2018 19:40, Segher Boessenkool wrote: > Hi! > > On Mon, Nov 26, 2018 at 02:02:49PM +, Torbjorn SVENSSON wrote: >> Attached is a small patch that, in case of inline assembler code, >> indicates that the function stack usage is uncertain due to inli

[PATCH] Added information about inline assembler in stack calculations (.su files)

2018-11-26 Thread Torbjorn SVENSSON
Hi, Attached is a small patch that, in case of inline assembler code, indicates that the function stack usage is uncertain due to inline assembler. The test suite are using "nop" as an assembler instruction on all targets, is this acceptable or is there a better way to test this? Patch has