[llvm] r280783 - Remove unnecessary call to getAllocatableRegClass

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 12:07:43 PDT 2016


On 9/30/2016 1:46 PM, Justin Bogner via llvm-commits wrote:
> The assert I'm seeing is in the case where `OpRC != nullptr` but
> `ConstrainedRC == nullptr`. Quentin's suggestion would call
> getAllocatableClass(DstRC) in this case, whereas in the committed code
> we just go ahead and do MRI->createVirtualRegister(OpRC). The call to
> createVirtualRegister asserts because OpRC isn't allocatable.

This doesn't sound right. This implies that a register was created for 
the operand that is not compatible with the operand's class. Sounds like 
the problem is somewhere else.

-Krzysztof

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, 
hosted by The Linux Foundation


More information about the llvm-commits mailing list