Zce LLVM (for prototype)

[jjscheel]

Technical Group

Code Size Reduction TG

ratification-pkg

Code Size

Technical Liaison

Tariq Kurd

Task Category

Compilers/Toolchain

Task Sub Category

• gcc
• binutils
• gdb
• intrinsics
• Java
• KVM
• ld
• llvm
• Linux kernel
• QEMU
• Spike

Ratification Target

1Q2023

Statement of Work (SOW)

SOW link

SOW Signoffs: (delete those not needed)

• Task group liaison sign-off date:
• Development partner sign-off date:
• Toolchains & Runtimes sign-off date (if gcc, LLVM, binutils, gdb work):

Waiver

• Freeze
• Ratification

Pull Request Details

1. Zcmp 1 - merged
2. Zcmp 2 - merged
3. Zcmp 3 - merged
4. Zcmp 4 - merged
5. Zcmt 1 - merged
6. Zcmt 2 - merged
7. lld patch - open

[jjscheel]

Patches remain in review per 3/28 meeting. No action needed.

[jjscheel]

@lazyparser, would you kindly help me get this assigned to Chunyu?

[ChunyuLiao]

GitHub ID: ChunyuLiao, You can assign it to me, sorry for the late reply.

[jjscheel]

Thanks, @ChunyuLiao. Would you kindly also make sure that you've added it to your RISC-V Profile in the GitHub id field. Details are here: link

Also, if you're not a member of the DevPartners community, please join.

[jjscheel]

@ChunyuLiao, nevermind. I see that you've already done this. THANKS!!!

[ChunyuLiao]

The following patches are under review in llvm:
Zcmp https://reviews.llvm.org/D132819 and https://reviews.llvm.org/D134599
Zcmt https://reviews.llvm.org/D133863 and https://reviews.llvm.org/D134600

[jjscheel]

Thanks, @ChunyuLiao! I've bumped the projected completion date to next meeting, April 25.

[jjscheel]

Based on last week's discussion that reviews are still pending, I've bumped the Completion Projection date to next meeting (May 9) to match Next Status date.

[ChunyuLiao]

[RISCV] Add MC support of RISCV zcmt Extension https://reviews.llvm.org/D134600 has been merged

[jjscheel]

Thanks, @ChunyuLiao! Great news!!! 1 down 3 to go.

[jjscheel]

Removed from Agenda for this week.

[Xinlong-Wu]

[RISCV] Add MC support of RISCV zcmp Extension https://reviews.llvm.org/D132819 has been merged

[jjscheel]

Way to go! Just 2 more!!!

[ChunyuLiao]

Split zcmp https://reviews.llvm.org/D134599 into three separate patches:
https://reviews.llvm.org/D134599
https://reviews.llvm.org/D150416
https://reviews.llvm.org/D150415

[jjscheel]

Done. Thank you, @ChunyuLiao!

[ChunyuLiao]

Zcmp have been updated. Need to sync-up two weeks later :)

[ChunyuLiao]

All Zcmp patchs have been merged.

[jjscheel]

Marking as done. Thanks, @ChunyuLiao! Nice work!!!

[ChunyuLiao]

Sorry Jeff. There is only one zcmt not merged and it is the 6th patch https://reviews.llvm.org/D134600.

[ChunyuLiao]

Thanks, @ChunyuLiao. Have you received any feedback on your latest patch?

No feedback. I'm guessing there won't be any feedback for the next month.

[jrtc27]

Your last update on the patch was 7th September. If you want people to review it you should ping the patch (adhering to https://llvm.org/docs/CodeReview.html#code-reviews-speed-and-reciprocity) and possibly them specifically, otherwise it drops off their radar.

[ChunyuLiao]

Your last update on the patch was 7th September. If you want people to review it you should ping the patch (adhering to https://llvm.org/docs/CodeReview.html#code-reviews-speed-and-reciprocity) and possibly them specifically, otherwise it drops off their radar.

Thanks, we're looking forward to your suggestions!

[ChunyuLiao]

No update for the last two weeks. thanks.

[ChunyuLiao]

The lld patch was just updated today

[ChunyuLiao]

Waiting for re-review.
We've done some testing and the results have been added to the comments, hope it helps.

[Xinlong-Wu]

Still in reviewing
Since the reviews.llvm.org has been turn to static, the patch is hard to be access and review patch.
So we are trying to move it to github PR.

[ChunyuLiao]

new pr: llvm/llvm-project#77884

[jjscheel]

Thanks, @ChunyuLiao. Added to the list in header.

[ChunyuLiao]

No new updates for the last 3 weeks.

[ChunyuLiao]

No updates for the last 2 weeks.

[ChunyuLiao]

Do more discussions and updated zcmt last week.

[ChunyuLiao]

no update.

[Xinlong-Wu]

No more update

[jjscheel]

@Xinlong-Wu, @ChunyuLiao, anything we can do to encourage review of the PR?
@cmuellner, are you aware of this PR and any problems here?

[Xinlong-Wu]

no update for pervious weeks.

currently， the reviewers are concerned about the efficiency(how much reduction we can get by current implementation) of zcmt can do.

It related to the linking time, so the reviewers would be very strict

[Xinlong-Wu]

no update

[Xinlong-Wu]

no more update