| Anonymous | Login | Signup for a new account | 2013-05-26 04:26 CEST | ![]() |
| Main | My View | View Issues | Change Log | Roadmap |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | |||||||
| 0005795 | OCaml | OCaml backend (code generation) | public | 2012-10-18 15:06 | 2012-12-18 13:21 | |||||||
| Reporter | chambart | |||||||||||
| Assigned To | lefessan | |||||||||||
| Priority | normal | Severity | feature | Reproducibility | always | |||||||
| Status | resolved | Resolution | fixed | |||||||||
| Platform | OS | OS Version | ||||||||||
| Product Version | 4.00.1 | |||||||||||
| Target Version | Fixed in Version | 4.01.0+dev | ||||||||||
| Summary | 0005795: [patch] Generate sqrtsd opcode instead of external call to sqrt on amd64 | |||||||||||
| Description | The sqrt function can be optimised by calling directly the processor instruction, avoiding a call. | |||||||||||
| Additional Information | The patch was not tested on windows | |||||||||||
| Tags | No tags attached. | |||||||||||
| Attached Files | ||||||||||||
Notes |
|
|
(0008469) lefessan (developer) 2012-11-09 14:27 |
Applied in trunk at revision r13086. |
|
(0008626) frisch (developer) 2012-12-18 11:48 |
Has anyone done any performance comparison? I'm curious about the performance gains to be expected, to see if it's worth the trouble extending the 32-bit (x87) backend as well (this backend is still very much useful for Windows...). |
|
(0008627) chambart (reporter) 2012-12-18 13:21 |
As far as I know the x87 backend is already doing that. See i386/selection.ml line 240. |
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-10-18 15:06 | chambart | New Issue | |
| 2012-10-18 15:06 | chambart | File Added: 0001-Generate-sqrtsd-instruction-on-amd64.patch | |
| 2012-11-09 14:27 | lefessan | Note Added: 0008469 | |
| 2012-11-09 14:27 | lefessan | Status | new => resolved |
| 2012-11-09 14:27 | lefessan | Fixed in Version | => 4.01.0+dev |
| 2012-11-09 14:27 | lefessan | Resolution | open => fixed |
| 2012-11-09 14:27 | lefessan | Assigned To | => lefessan |
| 2012-12-18 11:48 | frisch | Note Added: 0008626 | |
| 2012-12-18 13:21 | chambart | Note Added: 0008627 | |
| Copyright © 2000 - 2011 MantisBT Group |