Disable PIC/PIE for MSP430 target by default.

Relocatable code generation is meaningless on MSP430, as the platform is too small to use shared libraries.

Patch by Dmitry Mikushev!

Differential Revision: https://reviews.llvm.org/D56927

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@352181 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Driver/ToolChains/MSP430.h b/lib/Driver/ToolChains/MSP430.h
index 61fd0e1455bd3d8ecf8307c90fec64f07d0c5837..b5308a8dd6875c899e0eb1c90bf08d27b053bc03 100644 (file)
--- a/lib/Driver/ToolChains/MSP430.h
+++ b/lib/Driver/ToolChains/MSP430.h
@@ -36,6 +36,10 @@ public:
                              llvm::opt::ArgStringList &CC1Args,
                              Action::OffloadKind) const override;
 
+  bool isPICDefault() const override { return false; }
+  bool isPIEDefault() const override { return false; }
+  bool isPICDefaultForced() const override { return true; }
+
 protected:
   Tool *buildLinker() const override;
 

diff --git a/test/CodeGen/msp430-reloc.c b/test/CodeGen/msp430-reloc.c
new file mode 100644 (file)
index 0000000..c69e8e6
--- /dev/null
+++ b/test/CodeGen/msp430-reloc.c
@@ -0,0 +1,29 @@
+// RUN: %clang -target msp430 -fPIC -S %s -o - | FileCheck %s
+
+// Check the compilation does not crash as it was crashing before with "-fPIC" enabled
+
+void *alloca(unsigned int size);
+
+// CHECK: .globl foo
+short foo(char** data, char encoding)
+{
+       char* encoding_addr = alloca(sizeof(char));
+       *encoding_addr = encoding;
+
+       char tmp3 = *encoding_addr;
+       short conv2 = tmp3;
+       short and = conv2 & 0xf;
+
+       switch (and)
+       {
+       case 0 :
+       case 4 :
+       case 10 :
+               return 1;
+       case 11 :
+               return 2;
+       }
+
+       return 0;
+}
+