<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - swr doesn't compile with current LLVM 7.0 snapshots" href="https://bugs.freedesktop.org/show_bug.cgi?id=107146#c3">Comment # 3</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - swr doesn't compile with current LLVM 7.0 snapshots" href="https://bugs.freedesktop.org/show_bug.cgi?id=107146">bug 107146</a> from <a class="email" href="mailto:alok.hota@intel.com" title="Alok Hota <alok.hota@intel.com>"> Alok Hota</a> <pre>(In reply to Bernhard Rosenkraenzer from <a href="show_bug.cgi?id=107146#c0">comment #0</a>) > createInstructionSimplifierPass() has been removed in LLVM. This is used in > a few places in SWR; looks like this can safely be removed because it is > always preceded by createInstructionCombiningPass() which seems to be a > superset of what createInstructionSimplifierPass() used to do. > > Fixing this just unmasks another problem though - > Intrinsic::x86_fma_vfmadd_ps_256 is gone. Not sure how to replace that > properly. I can confirm we're seeing this issue as well. (In reply to Philip Meulengracht from <a href="show_bug.cgi?id=107146#c2">comment #2</a>) > Created <a href="attachment.cgi?id=140731" name="attach_140731" title="proposed patch">attachment 140731</a> <a href="attachment.cgi?id=140731&action=edit" title="proposed patch">[details]</a> <a href='page.cgi?id=splinter.html&bug=107146&attachment=140731'>[review]</a> [review] > proposed patch > > See my attachment for a proposed patch - I encounted this error too and > removed the references to the vfmaddps, and I removed the SimplePass's due > to them being unnecessary. Please feel free to correct me or the patch if > it's wrong. > > Instead of the vfmaddps it will use mul add. Thanks for the patch! I'm looking into this now.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> <li>You are the assignee for the bug.</li> </ul> </body> </html>