[llvm] Fix stack layout for frames larger than 2gb (PR #84114)

Wesley Wiser via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 6 07:54:54 PST 2024


================
@@ -280,7 +280,7 @@ class MachineFrameInfo {
   /// setup/destroy pseudo instructions (as defined in the TargetFrameInfo
   /// class).  This information is important for frame pointer elimination.
   /// It is only valid during and after prolog/epilog code insertion.
-  unsigned MaxCallFrameSize = ~0u;
+  unsigned long MaxCallFrameSize = ~0ul;
----------------
wesleywiser wrote:

I wasn't entirely sure when to use `unsigned long`/`long` instead of `uint64_t`/`int64_t` so I tried to follow the convention of the code I was changing. 

Is there a preferred style I should be using across the whole patch or would you prefer I just update this instance? 

https://github.com/llvm/llvm-project/pull/84114


More information about the llvm-commits mailing list