Re: [PATCH] Replace sync builtins with atomic builtins

PING! On Mon, Nov 12, 2018 at 4:05 PM Janne Blomqvist wrote: > The old __sync builtins have been deprecated for a long time now in > favor of the __atomic builtins following the C++11/C11 memory model. > This patch converts libgfortran to use the modern __atomic builtins. > > At the same time I

Re: OpenACC ICV acc-default-async-var

Hi Thomas, actually the current version of the acc_get/set_default_async patch is combined into: https://gcc.gnu.org/ml/gcc-patches/2018-09/msg01426.html This patch you're referring here was a version from early 2017. I'll try to reply to the still applying comments here below. On 2018/11/18 10

Re: [PATCH] Fix aarch64_compare_and_swap* constraints (PR target/87839)

On 11/13/18 10:28 AM, Jakub Jelinek wrote: > - [(match_operand:GPI 2 "aarch64_plus_operand" "rn") ;; expect > + [(match_operand:GPI 2 "aarch64_plus_operand" "rIJ");; expect Ah, yes. That's my bug. There is a similar one just above wrt aarch64_compare_and_swap, and that one need

Re: [RS6000] Ignore "c", "l" and "h" for register preference

Morning! On Mon, Nov 19, 2018 at 05:06:07PM +1030, Alan Modra wrote: > On Tue, Nov 13, 2018 at 05:37:07AM -0600, Segher Boessenkool wrote: > > On Tue, Nov 13, 2018 at 01:35:42PM +1030, Alan Modra wrote: > > > This catches a few places where move insn patterns don't slightly > > > disparage CTR, LR

Re: [RS6000] Ignore "c", "l" and "h" for register preference

On Tue, Nov 13, 2018 at 05:37:07AM -0600, Segher Boessenkool wrote: > On Tue, Nov 13, 2018 at 01:35:42PM +1030, Alan Modra wrote: > > This catches a few places where move insn patterns don't slightly > > disparage CTR, LR and VRSAVE regs. Also fixes the doc for the rs6000 > > h constraint, and rem

Re: [C++ PATCH] Fix ICE in adjust_temp_type (PR c++/87506)

On Fri, Nov 16, 2018 at 4:26 PM Jakub Jelinek wrote: > I admit this is just a shot in the dark, but I don't see why > one couldn't adjust a type of EMPTY_CLASS_EXPR to EMPTY_CLASS_EXPR > with a different variant of the same type. Makes sense. > Or, should I drop that > && TYPE_MAIN_VARIANT (ty

[doc, committed] document interaction between -p/-pg and no_instrument_function attribute

Like the subject line says. I've checked in this patch for PR 40498. -Sandra 2018-11-18 Sandra Loosemore PR other/40498 gcc/ * doc/extend.texi (Common Function Attributes): Document that no_instrument_function applies to -p and -pg, too. * doc/invoke.texi (Instrumentation Options): Add

[PATCH, libphobos] Committed remove unused internal modules

Hi, This patch removes two x86-centric modules that are ignored as gdc doesn't implement the D_InlineAsm version condition. Bootstrapped and testsuite ran on x86_64-linux-gnu. Committed to trunk as r266256 -- Iain --- libphobos/ChangeLog: 2018-11-19 Iain Buclaw * src/Makefile.am: Remo

Re: [C++ Patch] Remove obsolete _vptr check (and fix location)

On Sat, Nov 17, 2018 at 5:01 AM Paolo Carlini wrote: > while I was working on some location issues I noticed this check which > seems obsolete to me and means that we unnecessarily reject kosher code: > indeed we don't test for it anywhere and neither ICC nor clang enforce > it. I also went throug

[patch, committed] Fix PR 88073

Hello world, using the combined superpowers of Captain Obvious and Lieutenant Simple, I have committed the attached patch, which fixes a regression, as r266251, after regression-testing. I will backport to 8 and 7 in the next few days. Regards Thomas 2018-11-18 Thomas Koenig

[PATCH] RISC-V: Pass -mno-relax through to assembler if supported

GCC will emit ".option (no)relax" in its outputted assembly, but when using it as an assembler driver, such as for preprocessed assembly, it's merely preprocessing and therefore will not generate this directive. Therefore we should pass -mno-relax on to the assembler if specified (and supported) as

Re: *Ping* Fix PR 70260, ICE on invalid

Hi Paul1 OK for trunk. Thanks for working on it. Committed as r266248. Thanks for the review! Regards Thomas