r364679 - Revert enabling frame pointer elimination on OpenBSD for now.

Brad Smith via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 28 12:57:51 PDT 2019


Author: brad
Date: Fri Jun 28 12:57:51 2019
New Revision: 364679

URL: http://llvm.org/viewvc/llvm-project?rev=364679&view=rev
Log:
Revert enabling frame pointer elimination on OpenBSD for now.

Modified:
    cfe/trunk/lib/Driver/ToolChains/Clang.cpp
    cfe/trunk/test/Driver/frame-pointer-elim.c

Modified: cfe/trunk/lib/Driver/ToolChains/Clang.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains/Clang.cpp?rev=364679&r1=364678&r2=364679&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains/Clang.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains/Clang.cpp Fri Jun 28 12:57:51 2019
@@ -534,19 +534,6 @@ static bool useFramePointerForTargetByDe
     return !areOptimizationsEnabled(Args);
   }
 
-  if (Triple.isOSOpenBSD()) {
-    switch (Triple.getArch()) {
-    case llvm::Triple::mips64:
-    case llvm::Triple::mips64el:
-    case llvm::Triple::ppc:
-    case llvm::Triple::x86:
-    case llvm::Triple::x86_64:
-      return !areOptimizationsEnabled(Args);
-    default:
-      return true;
-    }
-  }
-
   if (Triple.isOSLinux() || Triple.getOS() == llvm::Triple::CloudABI ||
       Triple.isOSHurd()) {
     switch (Triple.getArch()) {

Modified: cfe/trunk/test/Driver/frame-pointer-elim.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/frame-pointer-elim.c?rev=364679&r1=364678&r2=364679&view=diff
==============================================================================
--- cfe/trunk/test/Driver/frame-pointer-elim.c (original)
+++ cfe/trunk/test/Driver/frame-pointer-elim.c Fri Jun 28 12:57:51 2019
@@ -26,19 +26,6 @@
 // RUN:   FileCheck --check-prefix=NETBSD %s
 // NETBSD-NOT: "-momit-leaf-frame-pointer"
 
-// OpenBSD follows the same rules as Linux.
-// RUN: %clang -### -target x86_64-unknown-openbsd -S -O1 %s 2>&1 | \
-// RUN:   FileCheck --check-prefix=OPENBSD-OPT %s
-// RUN: %clang -### -target powerpc-unknown-openbsd -S -O1 %s 2>&1 | \
-// RUN:   FileCheck --check-prefix=OPENBSD-OPT %s
-// OPENBSD-OPT: "-momit-leaf-frame-pointer"
-
-// RUN: %clang -### -target x86_64-unknown-openbsd -S %s 2>&1 | \
-// RUN:   FileCheck --check-prefix=OPENBSD %s
-// RUN: %clang -### -target powerpc-unknown-openbsd -S %s 2>&1 | \
-// RUN:   FileCheck --check-prefix=OPENBSD %s
-// OPENBSD-NOT: "-momit-leaf-frame-pointer"
-
 // Darwin disables omitting the leaf frame pointer even under optimization
 // unless the command lines are given.
 // RUN: %clang -### -target i386-apple-darwin -S %s 2>&1 | \




More information about the cfe-commits mailing list