Hi all, After LLVM commit e88fc4046f70131e67411cee503201e3e9665891 [AMDGPU] SDWA: add support for GFX9 in peephole pass, MPV videos have their colors shifted significantly into the blue range. It is resolved with the commit directly before the above. I am using an RX 480 on kernel 4.11.6, mesa git, xserver git, and llvm git. Using other mpv outputs, such as vo=vaapi,vdpau,x11 or xv all have normal video. Let me know if you need any more information. Thanks, Sarnex
Created attachment 132151 [details] good R600_DEBUG=fs,vs,gs,ps,cs
Created attachment 132152 [details] bad R600_DEBUG=fs,vs,gs,ps,cs
As a final piece of information, if I compile LLVM with asserts enabled, I get the following assert mpv: /var/tmp/portage/sys-devel/llvm-9999/work/llvm-9999/lib/Target/AMDGPU/SIPeepholeSDWA.cpp:719: bool {anonymous}::SIPeepholeSDWA::convertToSDWA(llvm::MachineInstr&, const SDWAOperandsVector&): Assertion `AMDGPU::getNamedOperandIdx(SDWAOpcode, AMDGPU::OpName::omod) != -1' failed. Thanks, Sarnex
Yea, this commit also breaks (asserts) Unigine Valley on R9 285. I'll open a bug on llvm tracker and link back to this one.
(In reply to Andy Furniss from comment #4) > Yea, this commit also breaks (asserts) Unigine Valley on R9 285. > > I'll open a bug on llvm tracker and link back to this one. https://bugs.llvm.org/show_bug.cgi?id=33566 Though I couldn't add the author to CC :-(
(In reply to Andy Furniss from comment #5) > Though I couldn't add the author to CC :-( Oops ignore that bit - the author is addable, I was trying to add the llvm username which is different.
Fixed by https://github.com/llvm-mirror/llvm/commit/06ed4a14fdcacacc12b31f6316efaab5620ffe85
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.