This is an archive of the discontinued LLVM Phabricator instance.

[LoongArch] Add codegen support for multiplication operations
ClosedPublic

Authored by SixWeining on Jun 20 2022, 5:57 AM.

Diff Detail

Event Timeline

SixWeining created this revision.Jun 20 2022, 5:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 20 2022, 5:57 AM
SixWeining requested review of this revision.Jun 20 2022, 5:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 20 2022, 5:57 AM

Use ";;" prefix for non-RUN and non-CHECK comment line.

xen0n accepted this revision.Jun 21 2022, 8:19 PM

LGTM for generated LA64 assembly; I can't verify the LA32 assembly with gcc output due to gcc not currently supporting LoongArch ILP32 ABIs at all, but they look reasonable at a first glance. They should be results of generic codegen thus probably correct.

I'd like someone else to review the substantive changes (which should be trivial enough).

This revision is now accepted and ready to land.Jun 21 2022, 8:19 PM
MaskRay requested changes to this revision.Jun 22 2022, 5:22 PM
MaskRay added inline comments.
llvm/test/CodeGen/LoongArch/ir-instruction/mul.ll
2

Use llvm/utils/update_llc_test_checks.py if this checks so many lines.

This revision now requires changes to proceed.Jun 22 2022, 5:22 PM
SixWeining added inline comments.Jun 22 2022, 5:41 PM
llvm/test/CodeGen/LoongArch/ir-instruction/mul.ll
2

Thanks. Actually these check lines are generated by that python script with the first “Note:” line deleted manually. This is because LoongArch support for this script hasn’t been upstreamed since the lacking of function calls to write full test for this tool.

SixWeining added inline comments.Jun 23 2022, 5:36 AM
llvm/test/CodeGen/LoongArch/ir-instruction/mul.ll
2

Just now we request review for this tool in D128433. After that been merged we can directly use it.

Address @MaskRay's comment. Rebase to D128433 since we need to call update_llc_test_checks.py.

@MaskRay I have rebased to D128433 which add LoongArch support to update_llc_test_checks. Do you have any futher comments? Thanks.

This revision was not accepted when it landed; it landed in state Needs Review.Jul 8 2022, 2:21 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.

Rebase and merged first since this change has been approved previously. If @MaskRay and others have futher comments, we can revert or submit new revision. Thanks.