<div dir="ltr"><div class="gmail_quote">Hi Dylan,</div><div class="gmail_quote"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>          xswr)<br>
>              llvm_require_version $LLVM_REQUIRED_SWR "swr"<br>
> +            llvm_add_default_components "swr"<br>
> +            if test $LLVM_VERSION_MAJOR -ge 7; then<br>
> +                llvm_add_component "ipo" "swr"<br>
> +                llvm_add_component "ObjCARCOpts" "swr"<br>
<br>
Presumably this should be added to the meson build as well? There's an<br>
"llvm_modules" array around line 1170.<br></blockquote><div><br></div><div>I'm not quite sure how to add this.  The additional modules are only required for LLVM >= 7.  From the implementation in meson.build, it looks like the required llvm_modules are assembled and then used  the llvm dependency is searched for.  It seems a bit of chicken-egg problem in that I don't know if the extra modules are required until after I know the llvm version, but I need the required modules to dertime the llvm version.  Suggestions?</div><div><br></div><div>Thanks,<br></div><div>- Chuck</div><div><br></div></div></div>