[PATCH] D68979: [RISCV] Handle variable sized objects with the stack need to be realigned

Shiva Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 17 22:56:18 PDT 2019


shiva0217 added a comment.

In D68979#1713763 <https://reviews.llvm.org/D68979#1713763>, @apazos wrote:

> Shiva, I have tried a few workloads like EEMBC, SPEC2000/2006, perennial c++, plumhall. 
>  They don't seem to use variable length arrays nor allocas to test this patch.
>  Which test suite are you using?


Hi Ana,

Thanks for the work. I ran GCC testsuite and gcc.c-torture/compile/pr51354.c will trigger the patch.
I expect rv32e would have little more cases trigger the patch due to smaller stack alignment.
But the cases of variable-sized objects with realignment are not very often.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68979





More information about the llvm-commits mailing list