|Anonymous | Login | Signup for a new account||2016-07-01 06:36 CEST|
|Main | My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0006238||OCaml||OCaml backend (code generation)||public||2013-11-14 11:49||2016-06-09 14:23|
|Target Version||4.03.1+dev||Fixed in Version|
|Summary||0006238: please add an option to enable debug symbols only|
currently option -g mixes several things up - generation debug symbols, storing backtrace information, adding more precise bounds checking, disabling some optimizations (in bytecode).
For profiling one usually wants the same binary code that is run in production but with debug symbols. So the speed of the code is the same, but there is some extra information available on demand, and this build can be run on production at highest speed and then profiled live if any problem arises. Option -g makes this approach less attractive because of incurred performance cost (e.g. storing backtrace).
Can we please have another option to _only_ write out (native, i.e. DWARF) debugging info and perform no other changes to generated code?
|Tags||No tags attached.|
|I think this is a reasonable thing to do. Watch this space.|
While I think this is a reasonable feature, I think that in the long term the way to go is rather to provide more control to performance-conscious users of which raises will trigger backtrace recording (eg. the raise-variants work of Alain). Indeed, there should be a sweet spot where most exceptions used for control flow do not record any trace, only actually exceptional exceptions do, and you have a system that is both efficient (essentially as the no-trace one) and auditable in case of unplanned failure (a good reason to have *some* backtraces even in production). I can understand people disabling everything for absolute performances (and right now it's probably easier to implement), but long-term I think that's more of a minority use-case.
Of course if someone is ready to do the work, that's great in any case.
|Gabriel, your argument addresses backtraces only, not the bound-checking and disabled optimizations.|
> more precise bounds checking
Have you seen cases where keeping multiple call sites for caml_ml_array_bound_error yields a noticeable performance penalty?
> disabling some optimizations (in bytecode)
Since the request is to have a mode with maximal runtime performance (while keeping debugging information), I don't think that bytecode is relevant.
|In any case, the request is for adding debugging info to the executable, while guaranteeing no run-time overhead, and this seems quite reasonable to me.|
|Mark, is it time for a gentle ping?|
Please see the comment on the pull request
|2013-11-14 11:49||ygrek||New Issue|
|2013-11-18 17:28||shinwell||Assigned To||=> shinwell|
|2013-11-18 17:28||shinwell||Status||new => assigned|
|2013-11-18 17:29||shinwell||Note Added: 0010644|
|2013-11-18 17:29||shinwell||Status||assigned => acknowledged|
|2013-11-18 19:22||gasche||Note Added: 0010648|
|2014-02-19 16:50||doligez||Note Added: 0010951|
|2014-12-19 15:19||whitequark||Relationship added||related to 0006728|
|2014-12-23 16:06||frisch||Note Added: 0012963|
|2015-01-09 18:36||doligez||Note Added: 0013044|
|2015-01-09 18:36||doligez||Target Version||=> 4.02.3+dev|
|2015-07-10 18:54||doligez||Target Version||4.02.3+dev => 4.03.0+dev / +beta1|
|2016-04-18 23:13||gasche||Note Added: 0015800|
|2016-04-18 23:13||gasche||Target Version||4.03.0+dev / +beta1 => 4.03.1+dev|
|2016-06-09 14:23||shinwell||Note Added: 0015977|
|Copyright © 2000 - 2011 MantisBT Group|