Depends on D128898
Actually I don't quite get why this needs to be different just for LA64. Isn't the behavior for these two insns identical between LA32 and LA64?
movgr2fr is definitely not "FP to int". Maybe this comment should be fixed.
Yes, but we only use these two SDNodes for LA64. In LA32, we use bitconvert, such as (LoongArchFloat32InstrInfo. td, line 224-227)
Thanks for the suggestion, I will change that.
Thanks. I will change that.
Still not quite sure about the necessity of the _LA64 nodes/pats (is it again because of tblgen intricacies?) but the rest looks okay.
I'm not quite sure about what the "LA" suffix means. Aren't we obviously LoongArch? Or is this meaning "L(64-bit integer), Any FP precision"? This seems likely until I find both "w" and "l" ftintrz's are pattern-matched from this.
Yes, we want to reduce the intricacies of tblgen and use the name to identify this node only for LA64.
I had hoped to use "LA" for "LoongArch". Thanks for your reminding, I found this suffix redundant for the FTINT node, I will remove it.
I don't know if the LA64-specific DAG nodes could actually be removed for further simplicity, but the rest seems good to me. Someone could enlighten me on this.
What does anyext mean? Either this isn't meaningful and should be removed, or an explanatory comment should be around.
nit: "FP" or "Floating-point" for consistency.