<div dir="ltr">Yeah I just noticed that. Thanks!</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">~Craig</div></div>
<br><div class="gmail_quote">On Fri, Sep 23, 2016 at 8:30 AM, Krzysztof Parzyszek via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">MachineRegisterInfo::getRegCla<wbr>ss does take a virtual register and returns the TargetRegisterClass of that register. You're thinking about TargetRegisterInfo::getRegClas<wbr>s.<br>
<br>
-Krzysztof<div><div class="h5"><br>
<br>
On 9/23/2016 10:20 AM, Craig Topper via llvm-dev wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
This code or subtle variations of it appears in multiple targets. It<br>
tries to convert from a register to a register class using getRegClass,<br>
but getRegClass is really supposed to take a register class enum value<br>
and get the register class object for it. It doesn't convert a register<br>
to a class. In fact there's not always a single or canonical class for a<br>
given register.<br>
<br>
What is the right way to do this?<br>
<br>
    unsigned SrcReg = Reg + VA.getValNo();<br>
    unsigned DestReg = VA.getLocReg();<br>
    // Avoid a cross-class copy. This is very unlikely.<br>
    if (!MRI.getRegClass(SrcReg)->con<wbr>tains(DestReg))<br>
      return false;<br>
<br>
<br>
~Craig<br>
<br>
<br></div></div>
______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br><span class="HOEnZb"><font color="#888888">
</font></span></blockquote><span class="HOEnZb"><font color="#888888">
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation<br>
______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
</font></span></blockquote></div><br></div>