[PATCH] D57497: [RISCV] Passing small data limitation value to RISCV backend

Fri Mar 13 06:22:14 PDT 2020

lenary added a comment.

In D57497#1920870 <https://reviews.llvm.org/D57497#1920870>, @shiva0217 wrote:

> In D57497#1920097 <https://reviews.llvm.org/D57497#1920097>, @apazos wrote:
>
> > Thanks Shiva, I res-ynced and rebuilt the patch. It is working fine.
> >
> > I see there is a msmall-data-threshold flag used by Mips and Hexagon, and now we are adding a new flag msmall-data-limit. Should't we reuse the flag?
>
>
> Hi Ana, 
>  Thanks for trying the patch. msmall-data-limit is also a RISCV GCC flag, so I would recommend using the same flag as GCC.

How hard would it be to use the `-msmall-data-threshold` flag work if a `-msmall-data-limit` flag is not provided?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57497