[Mesa-dev] [PATCH] misc: move some member init into constructor initialisation lists

2016-09-06 Thread Jeremy Newton
This patch moves some member init into the class constructor initialisation lists. Rational: somewhat cleaner, better style, and considered to produce better performance. Note that I'm not exactly familiar with the patch submission project, or how to respond to mailing list threads. So, I

Re: [Mesa-dev] [PATCH] amdgpu/addrlib: use initialization list in addrobject

2017-07-25 Thread Jeremy Newton
sitory more difficult. It would be preferable to apply changes to >the internal addrlib first and Mesa would get them on the next addrlib >update. > >I personally don't bother cleaning up addrlib in Mesa except maybe >fixing warnings. > >Marek > >On Mon, Jul 24, 201

Re: [Mesa-dev] [PATCH] amdgpu/addrlib: use initialization list in addrobject

2017-07-24 Thread Jeremy Newton
Fair enough. Although from my tests with x86-64 GCC 6.3 (Fedora 25), it did produce a slightly smaller binary with this patch. With that said, I only used whatever the default optimization flags are, and I didn't do a diff on a disasm to see what actually changed. On Mon, Jul 24, 2017 at 1:03

[Mesa-dev] [PATCH] pipe-loader: use radeonsi for MM if amdgpu dri is used

2019-07-15 Thread Jeremy Newton
dropping the amdgpu-pro specific GBM implementation in favour of Mesa's libgbm. Signed-off-by: Jeremy Newton Signed-off-by: Marek Olšák Reviewed-by: Michel Dänzer --- src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium

Re: [Mesa-dev] [PATCH] pipe-loader: use radeonsi for MM if amdgpu dri is used

2019-07-15 Thread Jeremy Newton
Sorry about that, I've only used git email maybe three times in my life :) On Mon, Jul 15, 2019 at 10:00 AM Jeremy Newton wrote: > Thanks Marek and Michel, see the updated patch below: > > The amdgpu dri is used for the closed source AMD driver. Since this driver > does not implemen