[PATCH] D79084: [VE] Dynamic stack allocation

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 5 07:31:09 PDT 2020


arsenm added inline comments.


================
Comment at: llvm/include/llvm/IR/CallingConv.h:244-247
+    /// Calling convention used for NEC SX-Aurora VE llvm_grow_stack intrinsic
+    /// function.
+    VE_LLVM_GROW_STACK = 100,
+
----------------
Seems weird to me that this needs its own CC, or that it would have llvm in the name. This should be split into a separate patch, and needs some bitcode encoding tests


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79084





More information about the llvm-commits mailing list