PING^2
Hongyu Wang 于2020年7月17日周五 下午1:40写道：
>
> Update for SAPPHIRERAPIDS and PING
>
> Hongyu Wang 于2020年7月7日周二 上午11:24写道：
>
> >
> > Hi Kirill, could you help review this patch?
> >
> > Hongyu Wang 于2020年7月6日周一 上午9:58写道：
> > >
> > > Hi:
> > >
> > > This patch is about to support Intel Advanced
gcc/ChangeLog:
* genemit.c (main): Print 'split line'.
* Makefile.in (insn-emit.c): Define split count and file
---
gcc/Makefile.in | 10 ++
gcc/genemit.c | 87 -
2 files changed, 59 insertions(+), 38 deletions(-)
diff --git
在 2020年7月21日 +0800 PM2:55，Bin.Cheng ，写道：
> On Tue, Jul 21, 2020 at 11:14 AM Jojo wrote:
> >
> > gcc/ChangeLog:
> >
> > * genemit.c (main): Print 'split line'.
> > * Makefile.in (insn-emit.c): Define split count and file
> >
>
> Thanks for working one this, following comments are based on the
>
On Fri, Jul 17, 2020 at 2:28 PM Ian Lance Taylor wrote:
>
> This patch to libgo updates it to the Go 1.14.6 release. Bootstrapped
> and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline.
I have now committed this to the GCC 10 branch as well.
Ian
On Thu, Jul 23, 2020 at 9:53 PM Hongtao Liu wrote:
>
> On Thu, Jul 23, 2020 at 4:39 PM Jan Hubicka wrote:
> >
> > Hello,
> > sorry for taking so long to get to this.
> > > diff --git a/gcc/config/i386/i386-features.c
> > > b/gcc/config/i386/i386-features.c
> > > index 535fc7e981d..8f81d101382
On Thu, Jul 23, 2020 at 4:50 PM Martin Sebor wrote:
>
> On 7/23/20 2:18 PM, H.J. Lu wrote:
> > On Thu, Jul 23, 2020 at 1:14 PM Martin Sebor via Gcc-patches
> > wrote:
> >>
> >> On 7/22/20 2:23 AM, Rainer Orth wrote:
> >>> Hi Martin,
> >>>
> I have committed this change in r11-2231 after
We get an ICE when using -mpower10 and a -mcpu= value that is older
than power10. The -mpower10 option requires -mcpu=power10 or later.
The following patch enforces that.
This passed bootstrap and regtesting with no errors. Ok for trunk?
GCC 10 does not ICE on this test case, so I am not
On 7/23/20 2:18 PM, H.J. Lu wrote:
On Thu, Jul 23, 2020 at 1:14 PM Martin Sebor via Gcc-patches
wrote:
On 7/22/20 2:23 AM, Rainer Orth wrote:
Hi Martin,
I have committed this change in r11-2231 after Jeff approved it
off list last Thursday.
the new gcc.target/i386/memcpy-pr95886.c test
On Thu, Jul 23, 2020 at 05:47:28PM -0400, David Malcolm wrote:
> On Thu, 2020-07-23 at 12:28 -0400, Lewis Hyatt via Gcc-patches wrote:
> > Hello-
> >
> > The attached patch is complete including docs, but I tagged as RFC
> > because I am not sure if anyone will like it, or if the general
> >
On Wed, Jul 22, 2020 at 09:45:08PM +0200, Andrea Corallo wrote:
> > Should that actually be a sliding window, or should there actually just
> > not be more than N branches per aligned block of machine code? Like,
> > per fetch group.
> >
> > Can you not use ASM_OUTPUT_ALIGN_WITH_NOP (or
On Thu, 2020-07-23 at 12:28 -0400, Lewis Hyatt via Gcc-patches wrote:
> Hello-
>
> The attached patch is complete including docs, but I tagged as RFC
> because I am not sure if anyone will like it, or if the general
> reaction may
> be closer to recoiling in horror :). Would appreciate your
This patch by Clément Chigot adds AIX FAT library support to libgo.
This follows the same general idea as support in other GCC libraries
like libgcc and libsdc++. Committed to mainline.
Ian
83cc5e2b2f887d4bb2305658da382a65fdcaab29
diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE
The testglue object file gets interpreted as another input file,
changing the dump and aux output names in GCC unless it is protected
by -Wl, like board file-named extra inputs.
Refactor the code that modifies the board settings so that it can be
used to modify regular variables as well, and do
On 7/23/20 10:28 AM, Lewis Hyatt via Gcc-patches wrote:
Hello-
The attached patch is complete including docs, but I tagged as RFC
because I am not sure if anyone will like it, or if the general reaction may
be closer to recoiling in horror :). Would appreciate your thoughts,
please...
I don't
On Jul 23, 2020, Richard Biener wrote:
> On Thu, 23 Jul 2020, Zhanghaijian (A) wrote:
>> # This option restores naming of aux and dump output files
>> # after input files when multiple input files are named,
>> # instead of getting them combined with the output name.
> Does it actually do what
On Thu, Jul 23, 2020 at 02:29:00PM -0600, Jeff Law wrote:
> On Thu, 2020-07-23 at 15:19 -0500, Segher Boessenkool wrote:
> > On Thu, Jul 23, 2020 at 01:42:59PM -0600, Jeff Law wrote:
> > > On Thu, 2020-07-23 at 14:25 -0500, Pat Haugen via Gcc-patches wrote:
> > > > Disable -fcaller-saves by
On Thu, 2020-07-23 at 15:19 -0500, Segher Boessenkool wrote:
> Hi!
>
> On Thu, Jul 23, 2020 at 01:42:59PM -0600, Jeff Law wrote:
> > On Thu, 2020-07-23 at 14:25 -0500, Pat Haugen via Gcc-patches wrote:
> > > Disable -fcaller-saves by default.
> > >
> > > This patch turns off -fcaller-saves by
On Thu, Jul 23, 2020 at 1:14 PM Martin Sebor via Gcc-patches
wrote:
>
> On 7/22/20 2:23 AM, Rainer Orth wrote:
> > Hi Martin,
> >
> >> I have committed this change in r11-2231 after Jeff approved it
> >> off list last Thursday.
> >
> > the new gcc.target/i386/memcpy-pr95886.c test FAILs on 32-bit
Hi!
On Thu, Jul 23, 2020 at 01:42:59PM -0600, Jeff Law wrote:
> On Thu, 2020-07-23 at 14:25 -0500, Pat Haugen via Gcc-patches wrote:
> > Disable -fcaller-saves by default.
> >
> > This patch turns off -fcaller-saves by default so that IRA doesn't try
> > to use volatile regs for pseudos that are
On 7/22/20 2:23 AM, Rainer Orth wrote:
Hi Martin,
I have committed this change in r11-2231 after Jeff approved it
off list last Thursday.
the new gcc.target/i386/memcpy-pr95886.c test FAILs on 32-bit x86
(i386-pc-solaris2.11):
+FAIL: gcc.target/i386/memcpy-pr95886.c scan-rtl-dump-times
On Thu, 2020-07-23 at 14:25 -0500, Pat Haugen via Gcc-patches wrote:
> Disable -fcaller-saves by default.
>
> This patch turns off -fcaller-saves by default so that IRA doesn't try
> to use volatile regs for pseudos that are live across a call, which
> would then require LRA to save/restore the
Disable -fcaller-saves by default.
This patch turns off -fcaller-saves by default so that IRA doesn't try
to use volatile regs for pseudos that are live across a call, which
would then require LRA to save/restore the reg around the call.
Bootstrap/regtest on powerpc64le with no new regressions.
Another test case added to the bug since I posted the patch shows
that the no-warning bit set by the C++ front end is easily lost
during early folding, triggering the -Wnonull again despite
the suppression. The attached revision tweaks the folder in just
this one case to let the suppression take
Many thanks to Segher Boessenkool for the speedy review. The following
patch
has been committed after testing on x86_64-pc-linux-gnu. In the recent
patch
to simplify-rtx.c I had incorrectly extrapolated from AND distributing over
AND,
and IOR distributing over IOR, that XOR distributes over
Hi!
On Thu, Jul 23, 2020 at 04:56:14PM +0100, Jozef Lawrynowicz wrote:
> +static int
> +msp430_insn_cost (rtx_insn *insn, bool speed ATTRIBUTE_UNUSED)
> +{
> + int cost;
> +
> + if (recog_memoized (insn) < 0)
> +return 0;
> +
> + cost = get_attr_length (insn);
> + if
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the German team of translators. The file is available at:
https://translationproject.org/latest/gcc/de.po
(This file, 'gcc-10.2.0.de.po', has
On Thu, 23 Jul 2020, Martin Liška wrote:
> On 7/23/20 4:44 PM, Patrick Palka via Gcc-patches wrote:
> > Here's a rebased patch. Can another Vim user verify that this does the
> > right thing?
>
> Yes, it works correctly for me! I tested that before and after your change
> for
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'gcc' has been made available
to the language teams for translation. It is archived as:
Hello-
The attached patch is complete including docs, but I tagged as RFC
because I am not sure if anyone will like it, or if the general reaction may
be closer to recoiling in horror :). Would appreciate your thoughts,
please...
Currently, if a UTF-8 locale is detected, GCC changes the quote
_ITM_beginTransaction is a 'returns_twice' function that saves x30
on the stack as part of gtm_jmpbuf (that is passed down to
GTM_begin_transaction), but the saved x30 is also used for return.
The return path should be protected so we don't leave an
ldp x29, x30, [sp]
ret
gadget in the code,
This note is not used anywhere currently but it is supposed to mark
objects if the return address is protected with PAC on the stack.
Since lse.S only has leaf functions the return address is never
saved on the stack so we can add the note.
The note is only added if pac-ret is enabled because it
On Thu, 16 Jul 2020 15:53:54 +0200
Tobias Burnus wrote:
> [This is an OpenACC issue but I would not completely surprised if
> something similar could occur for OpenMP offloading as well.
> However, the patch is for an OpenACC-specific function.]
>
> This issue occurs for
To access the "n - 10"th element of "a" in this test, GCC will
generate the following code for msp430-elf with -mcpu=msp430x:
RLAM.W #1, R12
MOV.W a-3392(R12), R12
Since there aren't actually 100,000 elements in a, this means that
"a-3392" offset calculated by the linker can overflow,
The length of an insn can be used to calculate its cost, when optimizing for
size. When optimizing for speed, this is a good estimate, since the cycle cost
of an MSP430 instruction increases with its length.
>From e4c5f9c3f567489f89b41a0d96e321acb5d18152 Mon Sep 17 00:00:00 2001
From: Jozef
The length of MSP430 instructions is mostly just a function of the type and
number of operands. Setting the "type" attribute on all insns describes
the number of operands, and the position of the source and destination
operands.
In most cases, defaulting in the "length" and "extension" attribute
Costs of MSP430 instructions are mostly just a function of the type and number
of operands. In these cases, TARGET_RTX_COSTS just needs to examine the
operands to calculate the cost of the expression.
For more complicated operations where library helper functions are required,
if the cost cannot
The cycle and size cost of a MOV instruction in different addressing
modes can be used to calculate the TARGET_MEMORY_MOVE_COST relative to
TARGET_REGISTER_MOVE_COST.
>From c801a2851d47601218578c411854de9540486335 Mon Sep 17 00:00:00 2001
From: Jozef Lawrynowicz
Date: Thu, 16 Jul 2020 11:28:11
The following series of patches for MSP430 implement some of the target
macros used to determine the relative costs of operations.
To give an indication of the overall effect of these changes on
codesize, below are some size statistics collected from all the
executable files from execute.exp that
Hi Matthias!
On 2020-07-23T15:54:55+0200, Matthias Klose wrote:
> Trying to build a nvptx offload compiler on aarch64-linux-gnu
I have not yet heard any reports of ARM-host GCC builds with offloading
enabled: 32-bit ARM, or 64-bit aarch64.
In general, we've designed/implemented code offloading
On Thu, Jul 23, 2020 at 05:06:31PM +0200, Tobias Burnus wrote:
> Another simple-to-add feature as the main work was
> done in the middle-end for C/C++.
>
> (I do note that 'master taskloop [simd]' is not yet
> supported in gfortran.)
>
> OK?
LGTM, thanks.
Jakub
On 7/23/20 4:44 PM, Patrick Palka via Gcc-patches wrote:
Here's a rebased patch. Can another Vim user verify that this does the
right thing?
Yes, it works correctly for me! I tested that before and after your change
for ./libstdc++-v3/include/std/iomanip.
Please install the patch.
Martin
Another simple-to-add feature as the main work was
done in the middle-end for C/C++.
(I do note that 'master taskloop [simd]' is not yet
supported in gfortran.)
OK?
Tobias
-
Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany
Registergericht München
Hi,
Thanks a lot for the info.
> On Jul 23, 2020, at 5:07 AM, Martin Liška wrote:
>
> On 7/21/20 6:24 PM, Qing Zhao wrote:
>> 4.
>> Our company is waiting for this patch to be committed to upstream.
>
> Hello.
>
> Please note that patch review can sometimes take some time.
Yes, I
On Mon, 3 Feb 2020, Patrick Palka wrote:
> Currently this script doesn't set the indentation style for the standard
> library
> headers under libstdc++/ because they lack a file extension. But they do
> have a modeline, so the file type is still set appropriately by Vim. So by
> inspecting ,
This commit adds CUDA_Execute and CUDA_Global to the list of allowed
pragmas. It also implements basic validation of said pragmas.
gcc/ada/
* aspects.ads: Declare CUDA_Global as aspect.
* einfo.ads: Use Flag118 for the Is_CUDA_Kernel flag.
(Set_Is_CUDA_Kernel):
Trying to build a nvptx offload compiler on aarch64-linux-gnu, the libgomp tests
error out with
unrecognizable argument of option -foffload-abi
Passing that option goes a step further, hitting PR target/96265. Define that
hook, as it was done for rs6000 in 2015. Ok for the trunk?
Matthias
*
On Thu, Jul 23, 2020 at 4:39 PM Jan Hubicka wrote:
>
> Hello,
> sorry for taking so long to get to this.
> > diff --git a/gcc/config/i386/i386-features.c
> > b/gcc/config/i386/i386-features.c
> > index 535fc7e981d..8f81d101382 100644
> > --- a/gcc/config/i386/i386-features.c
> > +++
Access values should never designate unaliased components.
This new feature is documented in AI12-0027-1.
gcc/ada/
* sem_ch13.ads (Same_Representation): Renamed as
Has_Compatible_Representation because now the order of the arguments
are taken into account; its formals are
Hi.
There's one obvious patch that I'm going to commmit on behalf of Dong.
Martin
2020-07-21 Dong JianQiang
gcc/ChangeLog:
PR gcov-profile/96267
* gcov-io.c (gcov_open): enable if IN_GCOV_TOOL.
---
gcc/gcov-io.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
Add the capability to use the Write_* procedures in an environment where
you want to write debugging info but still use them to write to other
files (such a C source files).
[changelog]
* output.ads (Push_Output, Pop_Output): New procedures.
* output.adb (FD_Array, FD_Stack,
On Thu, Jul 23, 2020 at 02:42:25PM +0200, Tobias Burnus wrote:
> libomp: Add omp_depend_kind to omp_lib.{f90,h}
>
> gcc/fortran/ChangeLog:
>
> * intrinsic.texi (OMP_LIB_KINDS): Add omp_depend_kind.
>
> libgomp/ChangeLog:
>
> * configure.ac: Add OMP_DEPEND_KIND and OMP_INT128_SIZE.
Hi Thomas,
On 7/23/20 12:09 PM, Thomas Koenig wrote:
can you also update https://gcc.gnu.org/gcc-11/changes.html ?
do you mind if I put it on the backburner? I think there will be several
more OpenMP changes during GCC 11 and I hope that one can then write a
final status which will be:
As just discussed,* this patch adds omp_depend_kind
to Fortran's omp_lib.{h,f90}.
I fixed a problem in omp_lib.h (which as already fixed
in the module version) and I added a sizeof check,
which I missed when adding the other items previously.
OK?
Tobias
*
On 2020/7/23 04:30, Richard Sandiford wrote:
>
> I now realise the reason is that the starting mode is too wide.
> I think we should fix that by doing:
>
>FOR_EACH_MODE_IN_CLASS (new_mode_iter, MODE_INT)
> {
>…
>
> and then add:
>
>if (maybe_lt (GET_MODE_SIZE
On Thu, Jul 23, 2020 at 2:01 PM Roger Sayle wrote:
>
>
> On Thu, Jul 23, 2020 at 10:02 Richard Biener wrote:
> > Likewise for the existing
> >
> >+/* popcount(X) == 0 is X == 0, and related (in)equalities. */ (for
> >+(for popcount (POPCOUNT)
> > (for cmp (le eq ne gt)
> > rep (eq eq ne
On Thu, Jul 23, 2020 at 10:02 Richard Biener wrote:
> Likewise for the existing
>
>+/* popcount(X) == 0 is X == 0, and related (in)equalities. */ (for
>+(for popcount (POPCOUNT)
> (for cmp (le eq ne gt)
> rep (eq eq ne ne)
> (simplify
> (cmp (popcount @0) integer_zerop)
>
PING^3
On 6/24/20 11:09 AM, Martin Liška wrote:
PING^2
On 6/10/20 10:12 AM, Martin Liška wrote:
PING^1
On 5/25/20 3:10 PM, Martin Liška wrote:
On 5/21/20 4:53 PM, Martin Sebor wrote:
On 5/21/20 5:28 AM, Martin Liška wrote:
On 5/18/20 10:37 PM, Martin Sebor wrote:
I know there are some
On Thu, Jul 23, 2020 at 12:58 PM Richard Sandiford
wrote:
>
> Richard Biener via Gcc-patches writes:
> > On Thu, Jul 23, 2020 at 11:11 AM Marc Glisse wrote:
> >>
> >> On Thu, 23 Jul 2020, Marc Glisse wrote:
> >>
> >> > On Wed, 22 Jul 2020, Roger Sayle wrote:
> >> >
> >> >> Many thanks for the
Richard Biener via Gcc-patches writes:
> On Thu, Jul 23, 2020 at 11:11 AM Marc Glisse wrote:
>>
>> On Thu, 23 Jul 2020, Marc Glisse wrote:
>>
>> > On Wed, 22 Jul 2020, Roger Sayle wrote:
>> >
>> >> Many thanks for the peer review and feedback. I completely agree that
>> >> POPCOUNT
>> >> and
On 7/21/20 6:07 AM, Fangrui Song wrote:
If the value does not contain any path component separator (e.g. a
slash), the linker will be searched for using COMPILER_PATH followed by
PATH. Otherwise, it is either an absolute path or a path relative to the
current working directory.
--ld-path=
Hi Tobias,
can you also update https://gcc.gnu.org/gcc-11/changes.html ?
Best regards
Thomas
On 7/21/20 6:24 PM, Qing Zhao wrote:
4.
Our company is waiting for this patch to be committed to upstream.
Hello.
Please note that patch review can sometimes take some time.
You can still install the patch to your local branch and note that
the next major release will be released in about 9
On Thu, Jul 23, 2020 at 11:51:38AM +0200, Tobias Burnus wrote:
> Except for a few known bugs (and a bunch of unknown ones),
> I am not aware of any outstanding issues with OpenMP 4.5.
> Additionally, the openmp_version claims 201511 (= 4.5).
> Hence, state that 4.5 is supported – and OpenMP 5.0 is
On Thu, Jul 23, 2020 at 11:11 AM Marc Glisse wrote:
>
> On Thu, 23 Jul 2020, Marc Glisse wrote:
>
> > On Wed, 22 Jul 2020, Roger Sayle wrote:
> >
> >> Many thanks for the peer review and feedback. I completely agree that
> >> POPCOUNT
> >> and PARITY iterators simplifies things and handle the
Except for a few known bugs (and a bunch of unknown ones),
I am not aware of any outstanding issues with OpenMP 4.5.
Additionally, the openmp_version claims 201511 (= 4.5).
Hence, state that 4.5 is supported – and OpenMP 5.0 is
partially.
I did update the .texi file to match the current content
On Thu, 23 Jul 2020, Marc Glisse wrote:
On Wed, 22 Jul 2020, Roger Sayle wrote:
Many thanks for the peer review and feedback. I completely agree that
POPCOUNT
and PARITY iterators simplifies things and handle the IFN_ variants.
Is there a reason why the iterators cannot be used for this
On Thu, Jul 23, 2020 at 10:15 AM Marc Glisse wrote:
>
> On Wed, 22 Jul 2020, Roger Sayle wrote:
>
> > Many thanks for the peer review and feedback. I completely agree that
> > POPCOUNT
> > and PARITY iterators simplifies things and handle the IFN_ variants.
>
> Is there a reason why the
On Thu, 23 Jul 2020, Zhanghaijian (A) wrote:
> Hi,
>
>
>
> This is a simple fix for pr96230.
>
> When the dumpbase is an empty string, we should add check for !*dumpbase in
> process_command.
>
>
>
> The option -dumpbase "" is used gcc-defs.exp:
>
> # This option restores naming of aux
Thanks, Richard!
I think your suggestion is very good, so I made a new patch.
v2: at a high level handles -falign-foo=0 like -falign-foo
v1: at the target level overides the -falign-foo=0 option values
Obviously, v2 is better than v1. In addition, anthor option
to reject 0 that discussed in the
It's going to fix git commit 76641cd8b53128e1a962f1313ba75acf0855fd00,
where we not correctly print:
git gcc-verify 76641cd8b53128e1a962f1313ba75acf0855fd00 -p
Checking 76641cd8b53128e1a962f1313ba75acf0855fd00: OK
-- gcc/ChangeLog --
2020-07-15 Richard Biener
Revert:
Hello,
sorry for taking so long to get to this.
> diff --git a/gcc/config/i386/i386-features.c b/gcc/config/i386/i386-features.c
> index 535fc7e981d..8f81d101382 100644
> --- a/gcc/config/i386/i386-features.c
> +++ b/gcc/config/i386/i386-features.c
> @@ -2379,6 +2379,152 @@
On Wed, 22 Jul 2020, Roger Sayle wrote:
Many thanks for the peer review and feedback. I completely agree that POPCOUNT
and PARITY iterators simplifies things and handle the IFN_ variants.
Is there a reason why the iterators cannot be used for this one?
+(for popcount (BUILT_IN_POPCOUNT
Hi,
This is a simple fix for pr96230.
When the dumpbase is an empty string, we should add check for !*dumpbase in
process_command.
The option -dumpbase "" is used gcc-defs.exp:
# This option restores naming of aux and dump output files
# after input files when multiple input files are
GCC 10.2 released, committed to GCC 10 branch :)
On Thu, Jul 23, 2020 at 3:21 PM Kito Cheng wrote:
>
> Hi Jakub:
>
> Thanks for your review, committed with formatting fixes.
>
> Hi Richard:
>
> Thanks, I'll commit that after gcc 10.2 release :)
>
> On Wed, Jul 22, 2020 at 6:14 PM Jakub Jelinek
Hi Jakub:
Thanks for your review, committed with formatting fixes.
Hi Richard:
Thanks, I'll commit that after gcc 10.2 release :)
On Wed, Jul 22, 2020 at 6:14 PM Jakub Jelinek via Gcc-patches
wrote:
>
> On Wed, Jul 22, 2020 at 04:53:00PM +0800, Kito Cheng wrote:
> > --- a/gcc/asan.c
> > +++
Status
==
The GCC 10.2 release process has been completed and the GCC 10 branch
is now again open for regression and documentation fixes.
Quality Data
Priority # Change from last report
--- ---
P1
P2
76 matches
Mail list logo
