[PATCH] Make the __morestack function available to the JIT memory manager under Linux.

Philip Reames listmail at philipreames.com
Mon Dec 29 12:45:23 PST 2014


LGTM - If you can, include the appropriate header, but you can submit either way.  That's extremely minor.


================
Comment at: lib/ExecutionEngine/RTDyldMemoryManager.cpp:215
@@ +214,3 @@
+extern "C" void __morestack();
+#endif
+
----------------
Why do we need to forward declare this function, but not the other glibc functions used in getSymbolAddressInProccess?  Is there a header we could include instead?

http://reviews.llvm.org/D6788

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list