[PATCH] D134646: [AArch64] Add a target feature for AArch64StorePairSuppress

Thu Sep 29 00:12:57 PDT 2022

dmgreen added inline comments.

================
Comment at: llvm/lib/Target/AArch64/AArch64.td:1031

 def TuneTSV110 : SubtargetFeature<"tsv110", "ARMProcFamily", "TSV110",
                                   "HiSilicon TS-V110 processors", [
----------------
Allen wrote:
> dmgreen wrote:
> > Allen wrote:
> > > TSV110 target is also expected to suppress pair stores
> > OK good to know, I can change it. I am a little surprised it expected to be beneficial though. Do you have an example where it helped? Perhaps I am just running the wrong code to show where it is useful..
> I don't have small case, but according the history record, this will benefical to **649.fotonik3d_s** in spec2006.
Hmm. 649.fotonik3d_s seems to be a fortran benchmark in spec2017. I would not usually measure those. I will see if I can measure some numbers.

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

https://reviews.llvm.org/D134646