<div dir="ltr"><div>As a person not developing on GlobalISel, I can already play with it by setting the flag. ;-)</div><div><br></div>The main (and huge) benefit I see is that it will get tested by default. So, I think it's mainly a question of maturity - if my (non-GlobalISel) change breaks GlobalISel, how likely is it to be a bug in my code vs. a bug in GlobalISel?<div><br></div><div>Michael</div></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>