[PATCH] Enable EHABI by default on non-Darwin ARM targets

Anton Korobeynikov anton at korobeynikov.info
Tue Jan 28 09:14:30 PST 2014


  Otherwise - LGTM. Thanks for finalizing this! :)


================
Comment at: lib/Target/ARM/ARMAsmPrinter.cpp:1125
@@ -1124,2 +1124,3 @@
   // Emit unwinding stuff for frame-related instructions
-  if (EnableARMEHABI && MI->getFlag(MachineInstr::FrameSetup))
+  if (!Subtarget->isTargetMachO() && !DisableARMEHABI &&
+       MI->getFlag(MachineInstr::FrameSetup))
----------------
Maybe we'd explicitly check for EABI subtarget here?


http://llvm-reviews.chandlerc.com/D2627



More information about the llvm-commits mailing list