This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add tests for (and (add x, c1), (lshr y, c2))
ClosedPublic

Authored by hev on Jul 9 2023, 7:10 PM.

Details

Summary

Add tests for (and (add x, c1), (lshr y, c2)).

Signed-off-by: WANG Rui <wangrui@loongson.cn>

Diff Detail

Event Timeline

hev created this revision.Jul 9 2023, 7:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2023, 7:10 PM
hev requested review of this revision.Jul 9 2023, 7:10 PM
asb added a comment.Jul 9 2023, 11:17 PM

Thanks for splitting this out. I think adding RV64I to this test file as you did in a previous iteration does make sense, so I'd recommend adding an RV64I RUN line in this patch.

The test file here appears to be for “loop strength reduce” pass originally. Maybe this should be a new test file?

hev updated this revision to Diff 538520.Jul 10 2023, 12:11 AM
  1. Move to a new test file.
  2. Add RV64I to test file.
hev added a reviewer: RKSimon.Jul 23 2023, 8:53 PM

Friendly ping.

This revision is now accepted and ready to land.Jul 23 2023, 9:34 PM
This revision was landed with ongoing or failed builds.Jul 24 2023, 1:52 AM
This revision was automatically updated to reflect the committed changes.