Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D141697

[P10] Fix the implementation for BRH
ClosedPublic
Actions

Authored by lei on Jan 13 2023, 8:09 AM.

Tags

Subscribers

Details

Reviewers

power-llvm-team
amyk
maryammo

Commits

rGee559b21b961: [P10] Fix the implementation for BRH

Summary

Fixes the patterns for the brh instruction to include a clrldi when emitted.

Diff Detail

Repository: rG LLVM Github Monorepo

Unit TestsFailed
View All

	Time	Test
	60,240 ms	x64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/non-overloaded::vloxseg.c
	60,200 ms	x64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/non-overloaded::vluxseg.c
	60,200 ms	x64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/overloaded::vloxseg.c
	60,280 ms	x64 debian > Clang.CodeGen/RISCV/rvv-intrinsics-autogenerated/policy/overloaded::vluxseg.c

Event Timeline

lei created this revision.Jan 13 2023, 8:09 AM

Herald added a project: Restricted Project. · View Herald TranscriptJan 13 2023, 8:09 AM

Herald added subscribers: hiraditya, nemanjai. · View Herald Transcript

lei requested review of this revision.Jan 13 2023, 8:09 AM

Herald added a project: Restricted Project. · View Herald TranscriptJan 13 2023, 8:09 AM

LGTM

LGTM as well.

This revision is now accepted and ready to land.Jan 13 2023, 8:21 AM

Harbormaster completed remote builds in B207652: Diff 489022.Jan 13 2023, 9:26 AM

This revision was landed with ongoing or failed builds.Jan 16 2023, 11:53 AM

Closed by commit rGee559b21b961: [P10] Fix the implementation for BRH (authored by lei). · Explain Why

This revision was automatically updated to reflect the committed changes.

lei added a commit: rGee559b21b961: [P10] Fix the implementation for BRH.

Revision Contents
Changeset List

Path

Size

llvm/

lib/

Target/

PowerPC/

8 lines

test/

CodeGen/

PowerPC/

5 lines

Diff 489022

llvm/lib/Target/PowerPC/PPCInstrP10.td

Loading...

llvm/test/CodeGen/PowerPC/p10-bswap.ll

Loading...