
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi,<div class=""><br class=""></div><div class="">I would like to get rid of the logic around disabling GlobalISel from the build.</div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">*** Context ***</div><div class=""><br class=""></div><div class="">So far, GlobalISel has been this framework built up on the side for about two years. Now, we are close to a point where having it being optional impedes our ability to implement the right things. For instance, checks that would fit naturally in the MachineVerifier are directly done during the execution of the related GISel generic passes to make sure we don’t end up with ifdefs all around of CodeGen.</div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">*** What Changed? ***</div><div class=""><br class=""></div><div class="">* GlobalISel is more mature</div><div class=""><br class=""></div><div class="">We now have a better idea of what the framework should look like and the scope of the changes what will happen on the CodeGen library.</div><div class=""><br class=""></div><div class="">* GlobalISel is getting to be the default for AArch64 O0</div><div class=""><br class=""></div><div class="">We are now on the verge of having GlobalISel being the default for AArch64 at O0. I would like to make sure it is not possible to build a configuration that wouldn’t support that default.</div><div class=""><br class=""></div><div class="">* GlobalISel is built by default since roughly 6 months</div><div class=""><br class=""></div><div class="">Unless people actively disabled GlobalISel while building LLVM, they already build it since end of January!</div><div class=""><a href="http://lists.llvm.org/pipermail/llvm-dev/2017-January/109029.html" class="">http://lists.llvm.org/pipermail/llvm-dev/2017-January/109029.html</a></div><div class=""><br class=""></div><div class=""> </div><div class="">*** What Do You Think? ***</div><div class=""><br class=""></div><div class="">As a remainder, the overhead of building GISel is about 1 min on a 8-core machine and the code size impact for llc is about 1 MB. Data form yesterday ToT attached r<span style="font-family: Menlo; font-size: 11px; background-color: rgb(255, 255, 255);" class="">307176</span>. (Format of the title of each section is <built target> <build type> <gisel or not>.)</div><div class=""><br class=""></div><div class="">As far as I know people don’t actively disable the building of GISel, so removing that feature shouldn’t be a big deal but I can be wrong.</div><div class=""><br class=""></div><div class="">Cheers,</div><div class="">-Quentin</div><div class=""></div></body></html>