This ensures the inline assembly register constraints are properly recognised in TargetLowering::getRegForInlineAsmConstraint.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
This looks pretty weird, that we match of node name, not the assembler name, but.... LGTM
Comment Actions
Maybe we can extend the logic in target lowering and will match assembler name of a register as well? This looks extremely reasonable.
Comment Actions
Yes, I think we should do this (maybe even drop matching on the node name completely?). Once I have some time, I'll try to prepare a patch and post it for review.