<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 29, 2015 at 4:53 PM Vasileios Kalintiris <<a href="mailto:Vasileios.Kalintiris@imgtec.com">Vasileios.Kalintiris@imgtec.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vkalintiris added a comment.<br>
<br>
Hi Eric,<br>
<br>
Thanks for your comments. I replaced -mips-fast-isel with sed and forgot to do the same for -fast-isel. It should be fine now.<br><br>
Comment at: test/CodeGen/Mips/Fast-ISel/sel1.ll:2<br>
@@ -1,3 +1,3 @@<br>
 ; RUN: llc < %s -march=mipsel -mcpu=mips32r2 -O2 -relocation-model=pic \<br>
-; RUN:          -fast-isel -mips-fast-isel -fast-isel-abort=1 | FileCheck %s<br>
+; RUN:          -fast-isel -fast-isel-abort=1 | FileCheck %s<br>
<br>
----------------<br>
echristo wrote:<br>
> Ditto.<br>
I believe that this one is required because of the -O2 level. Otherwise, I get the following assert error:<br>
<br>
```<br>
Assertion `(!EnableFastISelAbort || TM.Options.EnableFastISel) && "-fast-isel-abort > 0 requires -fast-isel"<br>
```<br>
<br>
Would you like me to change the level and whatever CHECK-lines are needed in this review? There are several things that we have to clean-up in FastISel and it's my intention to do that after the upcoming release.<br>
<br></blockquote><div><br></div><div>Do you actually mean to be using fast isel with optimization? It's fine if you are, but it's not a typical test route and you should probably comment it.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
================<br>
Comment at: test/CodeGen/Mips/emergency-spill-slot-near-fp.ll:2<br>
@@ -1,3 +1,3 @@<br>
 ; Check that register scavenging spill slot is close to $fp.<br>
-; RUN: llc -march=mipsel -O0 < %s | FileCheck %s<br>
+; RUN: llc -march=mipsel -O0 -fast-isel=false < %s | FileCheck %s<br>
<br>
----------------<br>
echristo wrote:<br>
> The typical thing is to just use llc -march here, but I gather optimization makes the testcase go away?<br>
That's right. However, we need the function's attributes too. This fine balance of options forces the OptLevelChanger to change the function's opt-level from -O2 to -O0. With -debug-only=isel, I get the following output:<br>
<br></blockquote><div><br></div><div>Weird on the optimization, I'm dubious here. Can you take a look at it?</div><div><br></div><div>(and yes, that's what optnone means)</div><div><br></div><div>Also, I meant that you could do the cleanup separately of the attribute move. If you need to change the attributes as part of the patch it's something to raise during review.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
```<br>
...<br>
Changing optimization level for Function main<br>
        Before: -O2 ; After: -O0<br>
....<br>
```<br>
<br>
I had to try by trial and error in order to get it right, after reading about this behaviour from lib/CodeGen/SelectionDag/SelectionDagISel.cpp and the comments at test/CodeGen/X86/dag-optnone.ll<br></blockquote><div><br></div><div>This  <br></div></div></div>