Re: patch to fix PR88157

2018-11-24 Thread Vladimir Makarov
On 11/24/2018 02:10 AM, Jeff Law wrote: On 11/23/18 3:04 PM, Vladimir Makarov wrote:   The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88157   The patch was successfully bootstrapped on x86 and x86-64 with GO and D.   Committed as rev. 266422. pr88157.patch

One more patch for PR88157

2018-11-24 Thread Vladimir Makarov
  Sorry,  my patch for pr88157 caused a lot of ICEs on different targets.  The problem that cost vector was not initialized on some targets for some modes (we use a lazy approach in cost calculations in RA).  Here is the patch which should fix the ICEs.   Committed as rev. 266435. Index:

Re: [patch, libgfortran] PR88052 - Format contravening f2008 constraint C1002 permitted

2018-11-24 Thread Jerry DeLisle
On 11/24/18 2:51 PM, Dominique d'Humières wrote: Hi Jerry, OK for trunk? Could you give me some time to post some comments in bugzilla? Sure, no rush. TIA Dominique

Re: [patch, libgfortran] PR88052 - Format contravening f2008 constraint C1002 permitted

2018-11-24 Thread Dominique d'Humières
Hi Jerry, > OK for trunk? Could you give me some time to post some comments in bugzilla? TIA Dominique

Re: Fix hashtable memory leak

2018-11-24 Thread François Dumont
Tests have shown a regression. I was building the _ReuseOrAllocNode instance too early, node ownership was transfered but was still owned by _Hashtable instance. This new version pass all tests. Ok to commit ? François On 11/19/18 10:23 PM, François Dumont wrote: There a memory leak when

[patch, libgfortran] PR88052 - Format contravening f2008 constraint C1002 permitted

2018-11-24 Thread Jerry DeLisle
Hi all, The attached patch adjusts the constraint of missing commas between format specifiers by only allowing if std=legacy is given at compile time. Several existing test cases are adjusted to be standard conforming. Regression tested on x86-64-linux-gnu. OK for trunk? (I will include the a

[PATCH, libstdc++] Uniform container erasure for c++20.

2018-11-24 Thread Ed Smith-Rowland
All, I's very late but uniform container erasure is, I think, the last little tidbit to graduate from fundamentals/v2 to std at the last meeting.  I think it would be a shame not to nudge this into gcc-9.  The routines are very short so I just copied them. Ditto the testcases (with

Re: LRA patch for PR87718

2018-11-24 Thread Christophe Lyon
On Thu, 22 Nov 2018 at 18:30, Vladimir Makarov wrote: > >The following patch fixes > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87718 > >The patch adds a special treatment for moves with a hard register in > register cost and class calculation. > >The patch was bootstrapped and

[PATCH, d] Fix IdentityExp comparison for complex floats

2018-11-24 Thread Johannes Pfau
Currently all identity comparisons for complex types (c1 is c2) return true. This is caused by the rrent implementation being only correct for real types, so this adds new code for the complex type cases. Fixes https://bugzilla.gdcproject.org/show_bug.cgi?id=309 Tested on x86_64-linux-gnu and on

[PATCH PR d/87866] Committed use lrealpath to get canonical name

2018-11-24 Thread Iain Buclaw
Hi, This patch fixes PR d/87866. The libiberty function is more portable than the one provided by the dmd front-end. I checked other places where FileName::canonicalName is used internally to dmd, but found none that would be used by us. Bootstrapped, ran D2 testsuite on x86_64-linux-gnu.