[PATCH] D12625: [llgo] Force exporting __morestack from llgoi

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 3 18:01:38 PST 2015

Previous message: [PATCH] D14313: Add a libLTO diagnostic handler that supports lto_get_error_message API
Next message: [PATCH] D14043: [bugpoint] Add a named metadata (+their operands) reducer
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

pcc added a comment.

LGTM

Another solution would be to have LLVMExecutionEngine contain a strong reference to `__morestack` conditional on whether the symbol is found at configure time. That seems kind of bad as well, since we try to avoid adding new configure-time checks. Up to you on which way to go though.


http://reviews.llvm.org/D12625

Previous message: [PATCH] D14313: Add a libLTO diagnostic handler that supports lto_get_error_message API
Next message: [PATCH] D14043: [bugpoint] Add a named metadata (+their operands) reducer
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list