<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
</head>
<body>
Thanks for reporting. I will fix this my tomorrow morning. As this is not yet used, it should not lead to incorrect behaviour, if this is a bug (it is groundwork for next patches).<br>
<br>
Thanks,<br>
Sjoerd.
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Puyan Lotfi via Phabricator <reviews@reviews.llvm.org><br>
<b>Sent:</b> 30 May 2019 20:56:57<br>
<b>To:</b> Simon Tatham; David Green; Sam Parker; Sjoerd Meijer; oliver.stannard@linaro.org<br>
<b>Cc:</b> puyan@puyan.org; mail@justinbogner.com; lebedev.ri@gmail.com; joker.eph@gmail.com; dexonsmith@apple.com; llvm-commits@lists.llvm.org<br>
<b>Subject:</b> [PATCH] D60696: [TableGen] New default operand "undef_tied_input".</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">plotfi added subscribers: bogner, plotfi.<br>
plotfi added a comment.<br>
<br>
I am seeing a build warning now. Is this a bug?<br>
<br>
/mnt/nvme0/llvm-project/llvm/utils/TableGen/GlobalISelEmitter.cpp: In member function ‘llvm::Error {anonymous}::GlobalISelEmitter::importDefaultOperandRenderers({anonymous}::action_iterator, {anonymous}::RuleMatcher&, {anonymous}::BuildMIAction&, llvm::DagInit*)
const’:<br>
/mnt/nvme0/llvm-project/llvm/utils/TableGen/GlobalISelEmitter.cpp:3816:9: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]<br>
if (DefaultDagOperator->getDef()->isSubClassOf("ValueType"))<br>
^~<br>
/mnt/nvme0/llvm-project/llvm/utils/TableGen/GlobalISelEmitter.cpp:3819:11: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’<br>
DefaultOp = DefaultDagOp->getArg(0);<br>
^~~~~~~~~<br>
<br>
@bogner<br>
<br>
<br>
Repository:<br>
rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
<a href="https://reviews.llvm.org/D60696/new/">https://reviews.llvm.org/D60696/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D60696">https://reviews.llvm.org/D60696</a><br>
<br>
<br>
<br>
</div>
</span></font></div>
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose,
or store or copy the information in any medium. Thank you.
</body>
</html>