[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