This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] [NFC] Add immediate version of the test for icmp Instruction
ClosedPublic

Authored by Miss_Grape on Mar 29 2022, 6:13 AM.

Diff Detail

Event Timeline

Miss_Grape created this revision.Mar 29 2022, 6:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 29 2022, 6:13 AM
Miss_Grape requested review of this revision.Mar 29 2022, 6:13 AM
craig.topper added inline comments.Mar 29 2022, 10:19 AM
llvm/test/CodeGen/RISCV/i32-icmp.ll
677

Why is there an extra underscore on the end of this test name?

llvm/test/CodeGen/RISCV/i64-icmp.ll
695

Same here

craig.topper accepted this revision.Mar 29 2022, 3:06 PM

LGTM if you remove those extra underscores in my other comments.

This revision is now accepted and ready to land.Mar 29 2022, 3:06 PM

Updata tests

llvm/test/CodeGen/RISCV/i32-icmp.ll
677

removed extra underscore

llvm/test/CodeGen/RISCV/i64-icmp.ll
695

done

This revision was landed with ongoing or failed builds.Mar 29 2022, 8:04 PM
This revision was automatically updated to reflect the committed changes.