<div dir="ltr">Thanks for pointing that out<div><br></div><div>I've sent a 4th patch, I've never compiled swr before, would you mind testing to see if it fixes things for you, it's using the llvm include rather than the llvm-c include</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, 1 Apr 2018 at 06:15 Vinson Lee <<a href="mailto:vlee@freedesktop.org">vlee@freedesktop.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Mar 31, 2018 at 5:08 PM, Mike Lothian <<a href="mailto:mike@fireburn.co.uk" target="_blank">mike@fireburn.co.uk</a>> wrote:<br>
> On 1 April 2018 at 00:03, Vinson Lee <<a href="mailto:vlee@freedesktop.org" target="_blank">vlee@freedesktop.org</a>> wrote:<br>
>><br>
>> A similar change is needed in<br>
>> src/gallium/drivers/swr/rasterizer/jitter/jit_pch.hpp.<br>
>><br>
>> Vinson<br>
><br>
> Hi<br>
><br>
> I'm not convinced that it does<br>
><br>
> Utils.h defines two functions LLVMAddLowerSwitchPass &<br>
> LLVMAddPromoteMemoryToRegisterPass. Whilst jit_pch.hpp does use<br>
> Scalar.h, it doesn't use either of the two functions in Utils.h so it<br>
> doesn't need to be included<br>
><br>
> Cheers<br>
><br>
> Mike<br>
<br>
I see this build errors with llvm 7.0.0svn. These errors are fixed if<br>
Utils.h is added to the list of includes.<br>
<br>
  CXX      rasterizer/jitter/libmesaswr_la-blend_jit.lo<br>
rasterizer/jitter/blend_jit.cpp:796:20: error: use of undeclared<br>
identifier 'createBreakCriticalEdgesPass'<br>
        passes.add(createBreakCriticalEdgesPass());<br>
                   ^<br>
rasterizer/jitter/blend_jit.cpp:799:20: error: use of undeclared<br>
identifier 'createPromoteMemoryToRegisterPass'<br>
        passes.add(createPromoteMemoryToRegisterPass());<br>
                   ^<br>
rasterizer/jitter/blend_jit.cpp:803:20: error: use of undeclared<br>
identifier 'createInstructionSimplifierPass'; did you mean<br>
      'createInstructionCombiningPass'?<br>
        passes.add(createInstructionSimplifierPass());<br>
                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>
                   createInstructionCombiningPass<br>
<br>
Vinson<br>
</blockquote></div>