[PATCH] D155988: Enable compact unwind in all darwin simulators

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 21 16:14:14 PDT 2023


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG62a1fbe9f7f9: Enable compact unwind in all darwin simulators (authored by jroelofs).

Changed prior to commit:
  https://reviews.llvm.org/D155988?vs=543029&id=543110#toc

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155988/new/

https://reviews.llvm.org/D155988

Files:
  llvm/lib/MC/MCObjectFileInfo.cpp
  llvm/test/CodeGen/X86/apple-simulator-compact-unwind.ll


Index: llvm/test/CodeGen/X86/apple-simulator-compact-unwind.ll
===================================================================
--- /dev/null
+++ llvm/test/CodeGen/X86/apple-simulator-compact-unwind.ll
@@ -0,0 +1,30 @@
+; RUN: llc -mtriple x86_64-apple-ios-simulator -filetype=obj -o - %s | \
+; RUN: llvm-objdump --macho --unwind-info - | \
+; RUN: FileCheck %s
+
+; RUN: llc -mtriple x86_64-apple-ios -filetype=obj -o - %s | \
+; RUN: llvm-objdump --macho --unwind-info - | \
+; RUN: FileCheck %s
+
+target datalayout = "e-m:o-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
+
+; Function Attrs: noinline nounwind optnone ssp uwtable
+define i32 @rdar104359594() #0 {
+entry:
+  ret i32 0
+}
+
+attributes #0 = { noinline nounwind optnone ssp uwtable "frame-pointer"="all" "min-legal-vector-width"="0" "no-builtin-calloc" "no-builtin-stpcpy" "no-trapping-math"="true" "stack-protector-buffer-size"="8" "target-cpu"="core2" "target-features"="+cmov,+cx16,+cx8,+fxsr,+mmx,+sahf,+sse,+sse2,+sse3,+ssse3,+x87" "tune-cpu"="generic" }
+
+!llvm.module.flags = !{!0, !1, !2, !3}
+!llvm.ident = !{!4}
+
+!0 = !{i32 1, !"wchar_size", i32 4}
+!1 = !{i32 8, !"PIC Level", i32 2}
+!2 = !{i32 7, !"uwtable", i32 2}
+!3 = !{i32 7, !"frame-pointer", i32 2}
+!4 = !{!"clang"}
+
+; Check that we generate compact unwind for simulators, which have always
+; supported it.
+; CHECK: Contents of __compact_unwind section:
Index: llvm/lib/MC/MCObjectFileInfo.cpp
===================================================================
--- llvm/lib/MC/MCObjectFileInfo.cpp
+++ llvm/lib/MC/MCObjectFileInfo.cpp
@@ -48,6 +48,10 @@
   if (T.isiOS() && T.isX86())
     return true;
 
+  // The rest of the simulators always have it.
+  if (T.isSimulatorEnvironment())
+    return true;
+
   return false;
 }
 
@@ -62,7 +66,8 @@
       SectionKind::getReadOnly());
 
   if (T.isOSDarwin() &&
-      (T.getArch() == Triple::aarch64 || T.getArch() == Triple::aarch64_32))
+      (T.getArch() == Triple::aarch64 || T.getArch() == Triple::aarch64_32 ||
+      T.isSimulatorEnvironment()))
     SupportsCompactUnwindWithoutEHFrame = true;
 
   switch (Ctx->emitDwarfUnwindInfo()) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D155988.543110.patch
Type: text/x-patch
Size: 2168 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230721/ba2011aa/attachment.bin>


More information about the llvm-commits mailing list