[PATCH] D35874: [MIPS] Implement support for -mstack-alignment.

Simon Dardis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 5 11:55:16 PDT 2017


sdardis added inline comments.


================
Comment at: lib/Target/Mips/MipsSubtarget.cpp:167
+  else
+    stackAlignment = 8;
+
----------------
Can you introduce an assertion here that the ABI is O32? If EABI support is ever brought back or another MIPS abi is introduced, I'd prefer debug builds complain loudly.


https://reviews.llvm.org/D35874





More information about the llvm-commits mailing list