[PATCH] D56925: Do not use frame pointer by default for MSP430

[Dmitry Mikushin via Phabricator via cfe-commits]

dmikushin created this revision.
Herald added a subscriber: cfe-commits.
dmikushin added a reviewer: asl.

> 3.3.9 Frame Pointer
>  MSP430 does not use a frame pointer. This effectively limits a single call frame to 0x7fff bytes, which is the minimum SP offset supported by any instruction

We do allow user to manually enable frame pointer. GCC toolchain uses the same behavior.


Repository:
  rC Clang

https://reviews.llvm.org/D56925

Files:
  lib/Driver/ToolChains/Clang.cpp


Index: lib/Driver/ToolChains/Clang.cpp
===================================================================
--- lib/Driver/ToolChains/Clang.cpp
+++ lib/Driver/ToolChains/Clang.cpp
@@ -519,10 +519,10 @@
   case llvm::Triple::xcore:
   case llvm::Triple::wasm32:
   case llvm::Triple::wasm64:
+  case llvm::Triple::msp430:
     // XCore never wants frame pointers, regardless of OS.
     // WebAssembly never wants frame pointers.
     return false;
-  case llvm::Triple::msp430:
   case llvm::Triple::riscv32:
   case llvm::Triple::riscv64:
     return !areOptimizationsEnabled(Args);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56925.182570.patch
Type: text/x-patch
Size: 583 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190118/f4062250/attachment.bin>