[PATCH] D134168: [RISCV] Make preferred alignment of PointerArgs for MemIntrinsic

Jiejie Rong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 19 19:42:42 PDT 2022


JojoR added a comment.

In D134168#3801742 <https://reviews.llvm.org/D134168#3801742>, @jrtc27 wrote:

>> Set default preferred alignment for MemIntrinsic like memcpy according to arch32 or arch64,
>> it will improve performance.
>
> What are arch32 and arch64?
>
>> e.g. dhrystone with "-O2" boosts performance by 50% on arch RV32.
>
> On what implementation? Does this affect actually-useful benchmarks, not just dhrystone? I would assume so, but it'd be more useful to get numbers for meaningful benchmarks rather than ones people should've long since abandoned.

I think WORD_SIZE as preferred alignment is basic behavior to improve performance (use lw/sw instead of lb/sb), and dhrystone gets good performance benefit from this patch,
some other useful benchmarks are up slightly (< 5%), I do not list here so.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134168/new/

https://reviews.llvm.org/D134168



More information about the llvm-commits mailing list