Generated code does not preserve the frame pointer and thus stack unwinding does not work when we crash in generated functions. We should preserve the frame pointers to make debugging easier. Kudu's experience and our own in
IMPALA-4132 suggest that the performance hit will be tolerable (but we still should do a proper perf evaluation).
Here's Kudu's way of preserving the frame pointers: module_builder.cc:294.
Thanks tlipcon for the suggestion.