[Bug 90056] Unigine Valley regression since radeon/llvm: Run LLVM's instruction combining pass - bugzilla-daemon

From: bugzilla-daemon@freedesktop.org
To: dri-devel@lists.freedesktop.org
Subject: [Bug 90056] Unigine Valley regression since radeon/llvm: Run LLVM's instruction combining pass
Date: Wed, 29 Apr 2015 02:06:43 +0000	[thread overview]
Message-ID: <bug-90056-502-MEx2z7FAmw@http.bugs.freedesktop.org/> (raw)
In-Reply-To: <bug-90056-502@http.bugs.freedesktop.org/>

[-- Attachment #1.1: Type: text/plain, Size: 1108 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=90056

--- Comment #12 from Tom Stellard <tstellar@gmail.com> ---
(In reply to Andy Furniss from comment #10)
> (In reply to Grigori Goronzy from comment #9)
> > Created attachment 115409 [details] [review] [review]
> > Full shader
> > 
> > That does not help either. Seems to break the SSA somehow. Full shader that
> > triggers the bug attached.
> > 
> > The patch I posted earlier help with the the reduced testcase, but not with
> > the full shader. Both undef incoming values in phi nodes and undef branch
> > conditions cause problems in different ways.
> 
> With the second patch + valley I get -
> 
> valley_x64: LiveVariables.cpp:114: void
> llvm::LiveVariables::MarkVirtRegAliveInBlock(llvm::LiveVariables::VarInfo &,
> llvm::MachineBasicBlock *, llvm::MachineBasicBlock *,
> std::vector<MachineBasicBlock *> &): Assertion `MBB != &MF->front() &&
> "Can't find reaching def for virtreg"' failed.

Did test with only the patch from comment #8 or did you test with both patches?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 2467 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel