Re: [Mesa-dev] PATCH: R600/SI: Enable the verifier on most lit tests

2013-10-08 Thread Vincent Lejeune
3rd patch is reviewed-by:Vincent Lejeune The first one >Subject: [PATCH 1/4] R600/SI: Mark the EXEC register as reserved > >This prevents the machine verifier from complaining about uses of >an undefined physical register. >--- > lib/Target/R600/SIRegisterInfo.cpp | 3 ++- > 1 file changed, 2 in

Re: [Mesa-dev] PATCH: R600/SI: Enable the verifier on most lit tests

2013-10-04 Thread Tom Stellard
On Fri, Oct 04, 2013 at 10:01:32AM -0700, Owen Anderson wrote: > > On Oct 3, 2013, at 5:36 PM, Tom Stellard wrote: > > > Unfortunately, we cannot enable the machine verifier on tests that have > > branches, because the way IF and ELSE instructions are selected leaves us > > with a copy instructi

Re: [Mesa-dev] PATCH: R600/SI: Enable the verifier on most lit tests

2013-10-04 Thread Owen Anderson
On Oct 3, 2013, at 5:36 PM, Tom Stellard wrote: > Unfortunately, we cannot enable the machine verifier on tests that have > branches, because the way IF and ELSE instructions are selected leaves us > with a copy instruction following the IF and ELSE terminators, which > violates one of the verif