View Issue Details

IDProjectCategoryView StatusLast Update
0002327SAS.ПланетаХотелка / Feature requestpublic02-09-2014 12:44
Reporterinfobriz Assigned Tovdemidov  
PrioritynormalSeveritytweakReproducibilityhave not tried
Status closedResolutionwon't fix 
Summary0002327: Конвертировал CityHash.c в CityHash.pas
Descriptionприложен также конвертированый тест от Google
Тест проходит. Cкорость работы немного упала.
TagsNo tags attached.
Attached Files
CityHash.pas.test.7z (56,141 bytes)

Activities

vdemidov

25-01-2014 19:41

manager   ~0013687

И зачем оно нужно, если скорость упала?

zed

13-02-2014 12:44

manager   ~0013773

Кстати, если есть такое уж сильное желание избавиться от этой dll, то можно ещё попробовать скомпилировать её в С++Builder и подключить как obj файлик к Delphi (так обычно подключается zlib во многих проектах). Скорость при этом упасть не должна.

vdemidov

13-02-2014 12:55

manager   ~0013774

Лично мне лишняя dll совсем не мешает. Вот если бы скорость росла, то это был бы аргумент. А так ИМХО овчинка выделки не стоит.

vdemidov

24-02-2014 16:06

manager   ~0013857

В общем, спасибо за помощь, может когда-нибудь и пригодится, но пока особого смысла я не вижу.

zed

02-09-2014 12:44

manager   ~0014604

Last edited: 02-09-2014 12:45

Ради интереса попробовал слинковать скомпиленный в С++Builder объектный файл и замерить скорость. Предварительно, правда пришлось сконвертировать код в простой Си, иначе Delphi тупо не видел экспортируемых функций.

По результатам тестов оказалось, что производительность таки упала почти в 2 раза по сравнению с dll. Скорость оказалась даже меньше, чем у паскалевского кода, который всего раза в 1,5 медленнее чем dll.

Единственное, что в сишном коде для билдера пришлось прописать не очень быстрые реализации bswap_32 и bswap_64, но не может быть, чтобы из-за этого скорость упала в 2 раза. Видно, у билдера оптимизатор плохо справляется со своей задачей.

Если вдруг кого заинтересует:
- код сложил в репо
- тестовые бинарники лежат в этом архиве

Issue History

Date Modified Username Field Change
25-01-2014 19:19 infobriz New Issue
25-01-2014 19:19 infobriz File Added: CityHash.pas.test.7z
25-01-2014 19:41 vdemidov Note Added: 0013687
30-01-2014 13:29 vdemidov Status new => feedback
13-02-2014 12:44 zed Note Added: 0013773
13-02-2014 12:55 vdemidov Note Added: 0013774
24-02-2014 16:06 vdemidov Note Added: 0013857
24-02-2014 16:06 vdemidov Status feedback => resolved
24-02-2014 16:06 vdemidov Resolution open => won't fix
24-02-2014 16:06 vdemidov Assigned To => vdemidov
24-02-2014 16:06 vdemidov Status resolved => closed
02-09-2014 12:44 zed Note Added: 0014604
02-09-2014 12:45 zed Note Edited: 0014604
08-08-2025 13:24 zed Category Хотелка => Хотелка / Feature request