<div dir="ltr">What is the right thing to do here?  Remove the pattern in .td, check for empty before inserting into the StringSet, or something else?</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 19, 2017 at 2:41 PM Krzysztof Parzyszek via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">kparzysz added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D37957#875513" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37957#875513</a>, @zturner wrote:<br>
<br>
> Can you apply this patch Krzysztof and see if you can figure out what the right thing to do is with the empty `StringRef`?<br>
<br>
<br>
It's because of this pattern that doesn't have any named variables (i.e. all names of leaf values are empty):<br>
`(intrinsic_void 5755:{ *:[iPTR] }, ECX:{ *:[i32] }, EAX:{ *:[i32] }, EBX:{ *:[i32] })`<br>
<br>
It corresponds to this pattern in .td, in case anyone's interested:<br>
<br>
  let Uses = [ ECX, EAX, EBX ] in {<br>
    def MWAITXrrr : I<0x01, MRM_FB, (outs), (ins), "mwaitx",<br>
                    [(int_x86_mwaitx ECX, EAX, EBX)], IIC_SSE_MWAITX>,<br>
                    TB, Requires<[ HasMWAITX ]>;<br>
  }<br>
<br>
<br>
<a href="https://reviews.llvm.org/D37957" rel="noreferrer" target="_blank">https://reviews.llvm.org/D37957</a><br>
<br>
<br>
<br>
</blockquote></div>