Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-10-25 Thread Timothy Arceri
On 26/10/18 10:21 am, Tom Stellard wrote: On 09/27/2018 10:01 PM, Timothy Arceri wrote: On 28/9/18 2:53 am, Tom Stellard wrote: On 09/24/2018 11:51 PM, Samuel Pitoiset wrote: On 9/25/18 6:46 AM, Timothy Arceri wrote: On 25/9/18 10:35 am, Marek Olšák wrote: Do you know what's broken in

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-10-25 Thread Tom Stellard
On 09/27/2018 10:01 PM, Timothy Arceri wrote: > On 28/9/18 2:53 am, Tom Stellard wrote: >> On 09/24/2018 11:51 PM, Samuel Pitoiset wrote: >>> >>> >>> On 9/25/18 6:46 AM, Timothy Arceri wrote: On 25/9/18 10:35 am, Marek Olšák wrote: > Do you know what's broken in LLVM? Or is it a problem

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-10-01 Thread Tom Stellard
On 09/27/2018 10:01 PM, Timothy Arceri wrote: > On 28/9/18 2:53 am, Tom Stellard wrote: >> On 09/24/2018 11:51 PM, Samuel Pitoiset wrote: >>> >>> >>> On 9/25/18 6:46 AM, Timothy Arceri wrote: On 25/9/18 10:35 am, Marek Olšák wrote: > Do you know what's broken in LLVM? Or is it a problem

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-27 Thread Timothy Arceri
On 28/9/18 2:53 am, Tom Stellard wrote: On 09/24/2018 11:51 PM, Samuel Pitoiset wrote: On 9/25/18 6:46 AM, Timothy Arceri wrote: On 25/9/18 10:35 am, Marek Olšák wrote: Do you know what's broken in LLVM? Or is it a problem with the ISA? I haven't actually dug any further. Adding Samuel to

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-27 Thread Tom Stellard
On 09/24/2018 11:51 PM, Samuel Pitoiset wrote: > > > On 9/25/18 6:46 AM, Timothy Arceri wrote: >> On 25/9/18 10:35 am, Marek Olšák wrote: >>> Do you know what's broken in LLVM? Or is it a problem with the ISA? >> >> I haven't actually dug any further. Adding Samuel to see if he remembers >>

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-27 Thread Marek Olšák
Since LLVM 7.0 has already been released: Reviewed-by: Marek Olšák Marek On Tue, Sep 25, 2018 at 12:46 AM Timothy Arceri wrote: > > On 25/9/18 10:35 am, Marek Olšák wrote: > > Do you know what's broken in LLVM? Or is it a problem with the ISA? > > I haven't actually dug any further. Adding

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-25 Thread Matt Turner
On Tue, Sep 25, 2018 at 8:50 AM Samuel Pitoiset wrote: > On 9/25/18 6:46 AM, Timothy Arceri wrote: > > On 25/9/18 10:35 am, Marek Olšák wrote: > >> Do you know what's broken in LLVM? Or is it a problem with the ISA? > > > > I haven't actually dug any further. Adding Samuel to see if he remembers

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-25 Thread Juan A. Suarez Romero
On Sat, 2018-09-22 at 12:38 +1000, Timothy Arceri wrote: > This ports the fix from 3d41757788ac. Both LLVM 7 & 8 continue > to have this problem. > > It fixes rendering issues in some menu and loading screens of > Civ VI which can be seen in the trace from bug 104602. > Thank you. Added this

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-25 Thread Samuel Pitoiset
On 9/25/18 6:46 AM, Timothy Arceri wrote: On 25/9/18 10:35 am, Marek Olšák wrote: Do you know what's broken in LLVM? Or is it a problem with the ISA? I haven't actually dug any further. Adding Samuel to see if he remembers more. However according to the original bug report this is a

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-24 Thread Timothy Arceri
On 25/9/18 10:35 am, Marek Olšák wrote: Do you know what's broken in LLVM? Or is it a problem with the ISA? I haven't actually dug any further. Adding Samuel to see if he remembers more. However according to the original bug report this is a regression when going from LLVM 6 to LLVM 7. I

Re: [Mesa-dev] [Mesa-stable] [PATCH] radeonsi: add a workaround for bitfield_extract when count is 0

2018-09-24 Thread Marek Olšák
Do you know what's broken in LLVM? Or is it a problem with the ISA? Marek On Fri, Sep 21, 2018 at 10:38 PM, Timothy Arceri wrote: > This ports the fix from 3d41757788ac. Both LLVM 7 & 8 continue > to have this problem. > > It fixes rendering issues in some menu and loading screens of > Civ VI