[PATCH] D91862: [VE] Correct types of return/argument values for getAdjustedFrameSize()

[Kazushi Marukawa via Phabricator via llvm-commits]

kaz7 created this revision.
kaz7 added reviewers: simoll, k-ishizaka.
kaz7 added projects: LLVM, VE.
Herald added subscribers: llvm-commits, hiraditya.
kaz7 requested review of this revision.

A getAdjustedFrameSize function may need to handle larger than 32 bits
integer, so change int to uint64_t.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D91862

Files:
  llvm/lib/Target/VE/VESubtarget.cpp
  llvm/lib/Target/VE/VESubtarget.h


Index: llvm/lib/Target/VE/VESubtarget.h
===================================================================
--- llvm/lib/Target/VE/VESubtarget.h
+++ llvm/lib/Target/VE/VESubtarget.h
@@ -72,7 +72,7 @@
   /// Given a actual stack size as determined by FrameInfo, this function
   /// returns adjusted framesize which includes space for register window
   /// spills and arguments.
-  int getAdjustedFrameSize(int stackSize) const;
+  uint64_t getAdjustedFrameSize(uint64_t stackSize) const;
 
   bool isTargetLinux() const { return TargetTriple.isOSLinux(); }
 };
Index: llvm/lib/Target/VE/VESubtarget.cpp
===================================================================
--- llvm/lib/Target/VE/VESubtarget.cpp
+++ llvm/lib/Target/VE/VESubtarget.cpp
@@ -47,7 +47,7 @@
       InstrInfo(initializeSubtargetDependencies(CPU, FS)), TLInfo(TM, *this),
       FrameLowering(*this) {}
 
-int VESubtarget::getAdjustedFrameSize(int frameSize) const {
+uint64_t VESubtarget::getAdjustedFrameSize(uint64_t frameSize) const {
 
   // VE stack frame:
   //


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91862.306660.patch
Type: text/x-patch
Size: 1044 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201120/df3af270/attachment.bin>