[PATCH] D50908: [PowerPC] Enable -funwind-tables by default on powerpc

[Stefan Pintilie via Phabricator via llvm-commits]

stefanp created this revision.
stefanp added reviewers: nemanjai, kbarton, hfinkel, sfertile, lei, syzaara.
Herald added a subscriber: chrib.

This is related to the bug

https://bugs.llvm.org/show_bug.cgi?id=32611

As shown in the above bug some functions with the nounwind attribute still require unwind tables. Using the uwtable attribute fixes that issue. 
At this point this patch changes the default behaviour on ppc linux to add the uwtable attribute on all functions.


https://reviews.llvm.org/D50908

Files:
  lib/Driver/ToolChains/Gnu.cpp


Index: lib/Driver/ToolChains/Gnu.cpp
===================================================================
--- lib/Driver/ToolChains/Gnu.cpp
+++ lib/Driver/ToolChains/Gnu.cpp
@@ -2367,7 +2367,9 @@
 }
 
 bool Generic_GCC::IsUnwindTablesDefault(const ArgList &Args) const {
-  return getArch() == llvm::Triple::x86_64;
+  return getArch() == llvm::Triple::x86_64 ||
+          getArch() == llvm::Triple::ppc64 ||
+          getArch() == llvm::Triple::ppc64le;
 }
 
 bool Generic_GCC::isPICDefault() const {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50908.161278.patch
Type: text/x-patch
Size: 503 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180817/fd3c7782/attachment.bin>