<div dir="ltr"><div>Hi Dylan,</div><div><br></div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think you could simplify this:<br>
<br>
> +  if dep_llvm.found() and with_gallium_swr and dep_llvm.version().version_compare('>= 7')<br>
> +    _llvm_2pass = true<br>
> +    llvm_modules += ['ipo', 'objcarcopts']<br>
> +  endif<br>
> +  if _llvm_2pass<br>
> +    dep_llvm = dependency(<br>
> +      'llvm',<br>
> +      version : _llvm_version,<br>
> +      modules : llvm_modules,<br>
> +      optional_modules : llvm_optional_modules,<br>
> +      required : _llvm_req,<br>
> +    )<br>
> +  endif <br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
if dep_llvm.found() and with_gallium_swr and dep_llvm.version().version_compare('>= 7')<br>
  llvm_modules += ['ipo', 'objcarcopts']<br>
  dep_llvm = dependency(<br>
    'llvm',<br>
    version : _llvm_version,<br>
    modules : llvm_modules,<br>
    optional_modules : llvm_optional_modules,<br>
    required : _llvm_req,<br>
  )<br>
endif<br>
with_llvm = dep_llvm.found()<br></blockquote><div><br></div><div>I could, and in fact, that exactly what I did initially.  But then I wanted to explicitly seperate the second pass from the decision to do so making it easy to allow other drivers to do  something similar if needed in the same spot.</div><div><br></div><div>- Chuck<br></div></div></div></div>