[PATCH] D64673: [X86] for split stack, not save/restore nested arg if unused

Cherry Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 16 08:28:56 PDT 2019


cherry added a comment.

Thanks for the review.

> We can obviously test this with Go -- do we know of any other easily usable testcases/applications that use the nested argument?

Closures do. The save/restore R10 <https://reviews.llvm.org/source/test-suite/> code path is used when a closure call splits the stack. For example, a recursive closure like https://play.golang.org/p/6t1xV8ThJuN .


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64673





More information about the llvm-commits mailing list