[PATCH] D87615: [X86] Fix stack alignment on 32-bit Solaris/x86

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 15 12:23:40 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Target/X86/X86Subtarget.cpp:270
+  else if (isTargetSolaris() && !In64BitMode)
+    stackAlignment = Align(4);
 
----------------
stackAlignment is initialized to 4 in the header, so `stackAlignment = Align(4)` here is a no-op.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87615



More information about the cfe-commits mailing list