Hello Gert,
your merge 'broke' LTO and then later on PGO compilation/linking.
I do generally compiling with '-Dgallium-drivers=r600,radeonsi,swrast'
for testing radeonsi and (your) r600 work. ;-)
After your merge I get several warnings in 'addrlib' with LTO and even a
compiler error (gcc
Those two patches are simple fixes to our svga driver and have been tested in
house for a while already.
So I thought it's ok to push the patches directly.
Thanks for reminding me of the MR practice, will keep that in mind for next
submits.
-Charmaine
On 2020-02-11 6:49 p.m., Charmaine Lee wrote:
>
> Those two patches are simple fixes to our svga driver and have been tested in
> house for a while already.
> So I thought it's ok to push the patches directly.
> Thanks for reminding me of the MR practice, will keep that in mind for next
>
I'm going to update the docs regarding patches and gitlab. It's kind of
a mess now.
-Brian
On 02/11/2020 03:03 AM, Michel Dänzer wrote:
Hi Charmaine,
it looks like you pushed this patch and another one directly to the main
Mesa repository master branch.
Pushing directly to the main
Hi Charmaine,
it looks like you pushed this patch and another one directly to the main
Mesa repository master branch.
Pushing directly to the main Mesa repository is no longer common
practice, and indeed discouraged, as it circumvents the pre-merge GitLab
CI pipeline and forfeits all other
