Re: [PATCH] rs6000: MMA built-in dies with incorrect sharing of tree nodes error

2020-09-01 Thread Peter Bergner via Gcc-patches
On 9/1/20 12:38 PM, Segher Boessenkool wrote: >> This passed bootstrap and regtesting on powerpc64le-linux with no >> regressions. >> Raji has also done some runtime testing of her MMA tests using this patch >> and they all showed no regressions either. Ok for trunk and GCC 10 after >> a couple o

Re: [PATCH] rs6000: MMA built-in dies with incorrect sharing of tree nodes error

2020-09-01 Thread Segher Boessenkool
Hi! On Tue, Sep 01, 2020 at 12:00:50PM -0500, Peter Bergner wrote: > When we expand our MMA built-ins into gimple, we (me!) erroneously reused > the accumulator memory reference for both the source input value as well as > the destination output value. This led to a tree sharing error. > The solu

[PATCH] rs6000: MMA built-in dies with incorrect sharing of tree nodes error

2020-09-01 Thread Peter Bergner via Gcc-patches
When we expand our MMA built-ins into gimple, we (me!) erroneously reused the accumulator memory reference for both the source input value as well as the destination output value. This led to a tree sharing error. The solution is to create separate memory references for the input and output values