[PATCH] D43916: Named VReg support for MIR

[Puyan Lotfi via llvm-commits]

Ah yeah that was a silly mistake. Thanks

PL


​Sent with ProtonMail Secure Email.​

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐

On March 29, 2018 11:48 PM, Francis Visoiu Mistrih via Phabricator via llvm-commits <llvm-commits at lists.llvm.org> wrote:

> thegameg added inline comments.
> 
> 
> ===================================
> 
> Comment at: lib/CodeGen/MIRParser/MIRParser.cpp:545
> 
> -   const VRegInfo &Info = *P.second;
> -   populateVRegInfo(Info, "");
>     
>     }
>     
> 
> The error will print: “Cannot determine class/bank of virtual register in function ‘func’”
> 
> Before this change P.first was the vreg ID which was a useful error message when you forget the reg class on a def.
> 
> Can you print the register ID if it’s a vreg and its name if it’s a named vreg?
> 
> Repository:
> 
> rL LLVM
> 
> https://reviews.llvm.org/D43916
> 
> llvm-commits mailing list
> 
> llvm-commits at lists.llvm.org
> 
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits