<div dir="ltr"><div dir="ltr">On Sat, 30 Oct 2021 at 17:15, Wang, Pengfei <<a href="mailto:pengfei.wang@intel.com">pengfei.wang@intel.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="EN-US" style="overflow-wrap: break-word;">
<div class="gmail-m_-1165746821762134812WordSection1">
<p class="MsoNormal"><span style="color:rgb(31,73,125)">SelectionDAG has many overloaded methods of “getNode”, some of which don’t need to specify the Flags argument. This is reasonable because only FP nodes need that. But it also easily results in losing the Flags
 in the FP nodes too.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:rgb(31,73,125)">Here’s an example <a href="https://reviews.llvm.org/D84518" target="_blank">
https://reviews.llvm.org/D84518</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:rgb(31,73,125)">This problem has been greatly improved since
<a href="https://reviews.llvm.org/D87361" target="_blank">https://reviews.llvm.org/D87361</a><u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:rgb(31,73,125)">I think we have the similar problem in MIR based optimizations, but I didn’t dig into it.</span></p></div></div></blockquote><div><br></div><div>This sounds like a somewhat large, but mostly mechanical problem to solve, correct?</div><div><br></div><div>Once we have a way to propagate the flags down to instruction selection, then we don't need the target info overriding IR semantics.</div><div><br></div><div>Andrew, is that what you're proposing originally? (Sorry if I'm slow to catch, I worried about changing the IR for front-ends but it seems it was misplaced).</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-US" style="overflow-wrap: break-word;"><div class="gmail-m_-1165746821762134812WordSection1"><div><div><div>
</div>
</div>
</div>
</div>
</div>

</blockquote></div></div>