<div dir="ltr">I'm in favor of building this by default. GlobalISel will eventually be the default, and people are actively working on it. We should iron out any build issues now to help with the transition. For example, if it comes with a substantial size hit, we should investigate and prioritize fixing that.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 13, 2017 at 5:54 PM, Quentin Colombet via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
Now, four backends (if I am counting right: X86, ARM, AArch64, AMDGPU) are working on bringing-up GlobalISel, I’d like to switch the default of the LLVM_BUILD_GLOBAL_ISEL variable in CMake, such that the framework gets built by default.<br>
<br>
** Impact of Flipping the Switch **<br>
<br>
* Upsides *<br>
<br>
For people developing on GlobalISel, it will:<br>
- Simplify the CMake command to type :)<br>
- Build/Test GlobalISel on all the LLVM bots<br>
<br>
For people not developing on GlobalISel, it will:<br>
- Test that GlobalISel still works with your changes (make check will test that for you)<br>
- Allow you to play with it!<br>
<br>
Basically flipping the default CMake setting will give access to all the ISel schemes that we have in LLVM, instead of just SDISel and FastISel.<br>
<br>
* Downsides *<br>
<br>
For people developing on GlobalISel, it will:<br>
- Leave the status as it is now, meaning that mainly only people working on GlobalISel look at the failures of GlobalISel specific bots<br>
<br>
For people not developing for GlobalISel, it will:<br>
- Increase the compile time since the GlobalISel framework and the related target specific parts will have to be built<br>
- Increase the size of the binary (depending on what backend you pull)<br>
- Require the setting of an additional CMake variable to disable it (-DLLVM_BUILD_GLOBAL_ISEL=OFF)<br>
<br>
<br>
What do people think?<br>
<br>
Thanks,<br>
-Quentin<br>
______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
</blockquote></div><br></div>