<html><body><p><font size="2">Hi Sander:</font><br><br><font size="2">It may be confusing, but it is not a bug. </font><br><font size="2">And you are right, if you are using the in-tree machinepipeliner, it won't have any effects,</font><br><font size="2">because there is NO In tree reference of this API at all.</font><br><br><font size="2">You can see the comments when </font><font face="Helvetica Neue">James</font><font size="2"> introduced this API in </font><a href="https://reviews.llvm.org/D67167"><u><font color="#0000FF">https://reviews.llvm.org/D67167</font></u></a><br><br><br><font size="2"><br>Best,<br><br>Jinsong Ji (ΌΝ½πΛΙ), PhD.<br><br>XL/LLVM on Power Compiler Development <br>E-mail: jji@us.ibm.com</font><br><br><img width="16" height="16" src="cid:1__=8FBB0FE8DFFEDF1A8f9e8a93df938690918c8FB@" border="0" alt="Inactive hide details for Sander via llvm-dev ---06/02/2020 03:47:53 PM---Hi all, I think there is a mistake in the machinepipe"><font size="2" color="#424282">Sander via llvm-dev ---06/02/2020 03:47:53 PM---Hi all, I think there is a mistake in the machinepipeliner interface. In the</font><br><br><font size="2" color="#5F5F5F">From: </font><font size="2">Sander via llvm-dev <llvm-dev@lists.llvm.org></font><br><font size="2" color="#5F5F5F">To: </font><font size="2">llvm-dev@lists.llvm.org</font><br><font size="2" color="#5F5F5F">Date: </font><font size="2">06/02/2020 03:47 PM</font><br><font size="2" color="#5F5F5F">Subject: </font><font size="2">[EXTERNAL] [llvm-dev] Machinepipeliner interface. shouldIgnoreForPipelining, actually not ignoring.</font><br><font size="2" color="#5F5F5F">Sent by: </font><font size="2">"llvm-dev" <llvm-dev-bounces@lists.llvm.org></font><br><hr width="100%" size="2" align="left" noshade style="color:#8091A5; "><br><br><br><tt><font size="2">Hi all,<br><br>I think there is a mistake in the machinepipeliner interface. In the <br>TargetInstrInfo.h in the class PipelinerLoopInfo there is a function <br>"bool shouldIgnoreForPipelining(const MachineInstr *MI)". The <br>description says that if this function returns true for a given <br>MachineInstr it will not be pipelined.<br><br>However in reality it is not ignored and is being considered for <br>pipelining. I implemented this function in my own backend, and put an <br>instruction there that I want to be ignored, but still this instruction <br>end up trying to be pipelined. I implemented the same way as in <br>PPCInstrInfo.cpp, and I think it has the same bug. Is this a bug, or am <br>I forgetting something?<br><br>Kind regards,<br><br>Sander Ruben<br><br>_______________________________________________<br>LLVM Developers mailing list<br>llvm-dev@lists.llvm.org<br></font></tt><tt><font size="2"><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a></font></tt><tt><font size="2"> <br><br></font></tt><br><br><BR>
</body></html>