Pawn compiler (3.10)

Модифицированная версия компилятора Pawn с исправлением множества багов.

Наиболее примечательные баги, которые были исправлены:

• Ошибка на строках кода длиннее 511 символов (лимит увеличен до 4095).
• Падение компилятора при использовании #emit sysreq.c, если указанная функция не использовалась ранее (с некоторых пор баг снова стал проявляться, однако новый патч #157 (https://github.com/Zeex/pawn/pull/157) уже ждёт принятия).
• Ошибка при использовании строк в тернарных выражениях.
• Падение компилятора при использовании #emit call.
• Падение компилятора из-за неправильного названия инструкции в #emit.
• Условная компиляция (#if/#elseif/#else/#endif) не работала с директивой #emit.
• Ошибка при передаче глобальной переменной в функцию до определения функции, из-за которой компилятор принимал переменную за неиспользуемую и удалял её из файла .amx (#131).

Нововведения:

• Добавлен ключ командной строки "-R", использование которого включает отображение отчёта о находящихся рекурсиях в скрипте (#121).
  
• Добавлена поддержка отрицательных чисел и чисел с плавающей запятой в #emit (#128, #133).
  
• Реализована возможность ограничивать видимость перечисления в пределах одного файла (удобно при создании инклудов).
  Пример:
  

  static enum eMyEnum
  {
      myEnumFieldA,
      myEnumFieldB,
      myEnumFieldC
  };

• Убрана автоматическая защита от повторного подключения включаемых файлов. Это было сделано из-за различий в работе такой защиты на разных ОС.
  Данную защиту можно включить, активировав режим совместимости с помощью флага "-Z" или директивы #pragma compat.
  
• Новые встроенные константы:
  • __line - номер текущей строки в исходном файле;
    
  • __PawnBuild - номер текущего билда компилятора;
    
  • __compat - флаг режима совместимости: 1, если включена компиляция в режиме совместимости (с ключом -Z), иначе 0;
    
  • __file - содержит строку с названием текущего исходного файла (например, "main.pwn" или "my_include.inc");
    
  • __date, __time - содержат дату и время компиляции соответственно (время запуска компилятора).
  
  
  
• Директива #pragma compat позволяет активировать режим совместимости, в котором включена автоматическая защита от повторного подключения файлов.
  
• Добавлена возможность показывать предупреждающие сообщения с помощью директивы #warning, по аналогии с уже существующей директивой #error, но без прерывания компиляции:
  Пример:
  

  #warning Эта строка будет выведена в отчёте компилятора

  
  Результат:
  
  Код:

  ---

  warning 237: user warning: Эта строка будет выведена в отчёте компилятора

  ---

• Также реализована многофункциональная директива #pragma warning, с помощью которой можно отключать предупреждения компилятора (#pragma warning disable <номер>), а также запоминать/восстанавливать состояние активации предупреждений (#pragma warning push/pop).
  Пример:
  

  #warning Эта строка будет выведена в отчёте компилятора
   
  #pragma warning push // Запоминаем состояние предупреждений
  #pragma warning disable 237 // Отключаем все пользовательские предупреждения (код 237)
  #warning Эта строка выведена не будет
  #pragma warning pop // Восстанавливаем прежнее состояние
   
  #warning Эта строка тоже будет выведена

  
  Результат:
  
  Код:

  ---

  warning 237: user warning: Эта строка будет выведена в отчёте компилятора
warning 237: user warning: Эта строка тоже будет выведена

  ---

• Директива #pragma naked отключает предупреждения вида "function must return a value". Отличие от #pragma warning disable в том, что #pragma naked действует только в пределах одного файла.
  Данная возможность может пригодиться при использовании #emit retn. Добавление return 1 в таких случаях тоже скроет предупреждение, но приведёт к генерации неиспользуемого (недостижимого) байткода.
  
• Добавлена прогрессивная инициализация для двухмерных массивов.
  Пример (взято отсюда):
  

  new a[5][10] = {{0, 1, 2, ...}, {0, 2, 4, ...}, ...};

  
  В полной записи данный массив будет выглядеть так:
  

  new a[5][10] = {
      {0, 1, 2, 3, 4, 5, 6, 7, 8, 9},
      {0, 2, 4, 6, 8, 10, 12, 14, 16, 18},
      {0, 3, 6, 9, 12, 15, 18, 21, 24, 27},
      {0, 4, 8, 12, 16, 20, 24, 28, 32, 36},
      {0, 5, 10, 15, 20, 25, 30, 35, 40, 45}
  };

• Также реализована возможность объявления четырёхмерных массивов (ранее был лимит в 3 измерения).
  Пример:
  

  new arr[3][1][2][3] =
  {
      { { { 1, 2, 3 }, { 4, 5, 6 } } },
      { { { 7, 8, 9 }, { 10, 11, 12 } } },
      { { { 13, 14, 15 }, { 16, 17, 18 } } }
  };

• Добавлен новый оператор __emit.
  С описанием работы данного оператора можно ознакомиться здесь: https://github.com/Zeex/pawn/pull/18...ment-323531746
  (Примечание (DC): В скором времени будет подготовлена русскоязычная статья по использованию оператора emit, я добавлю ссылку сюда по мере готовности.)
  
• Ускорена компиляция больших массивов, все элементы которых инициализированы одним и тем же значением.
  Теперь в ассемблерном листинге подобные массивы записываются как "dumpn <кол-во> <значение>".
  
• Теперь при возврате массивов неизвестного размера компилятор выдаёт ошибку.
  Пример:
  

  #include <a_samp>
   
  GetGenderName(gender_id)
  {
      static const gender_names[3][] = { {"male"}, {"female"}, {"undefined"} };
      if (0 == gender_id)
          return gender_names[0];
      if (1 == gender_id)
          return gender_names[1];
      return gender_names[2];
  }
   
  main()
  {
      for (new i = 0; i < 3; ++i)
      printf("#%d: %s", i, GetGenderName(i)); // error 092: functions may not return arrays of unknown size (symbol "GetGenderName")
  }

  
  Здесь компилятор выдаёт ошибку из-за того, что не знает размер последнего измерения массива gender_names. Чтобы устранить ошибку, достаточно лишь указать размер последнего измерения массива:
  

  static const gender_names[3][10] = { {"male"}, {"female"}, {"undefined"} };

  
  
• Значительно ускорена обработка исходного кода за счёт хранения глобальных идентификаторов (переменных, констант, функций) в хеш-таблице.
  
• Добавлен параметр командной строки "-E" для интерпретации предупреждений от компилятора как ошибок.
  
• Добавлена директива #pragma option, с помощью которой можно указывать параметры командной строки из скрипта.

Скачать: https://github.com/Zeex/pawn/releases
Исходный код на GitHub: https://github.com/pawn-lang/compiler

Источники информации: What's new, https://github.com/pawn-lang/compile... compiler bugs.
Статью подготовил: Daniel_Cortez

Отдельная благодарность ziggi за описания некоторых ошибок и нововведений.

Специально для Pro-Pawn.ru

Копирование данной статьи на других ресурсах без разрешения автора запрещено!

v3.10.1
Build 1

Начиная с этого релиза, в версии компилятора располагается номер сборки вместо даты, то есть "1" в "3.10.1" означает "build 1". Номер сборки будет увеличиваться с каждым релизом.

Изменения относительно предыдущей версии:

• Исправлена ошибка при передаче глобальной переменной в функцию до определения функции, что вызывало удаление переменной из AMX файла как неиспользуемую (#131)
• Добавлена опция -R использование которой включает отображение отчёта о находящихся рекурсиях в скрипте (#121)
• Добавлена поддержка отрицательных чисел и чисел с плавающей запятой в #emit (#128, #133)
• Добавлена константа __PawnBuild которая содержит номер сборки текущей версии компилятора (#132)

https://github.com/Zeex/pawn/releases

Ещё с прошлой недели понемногу готовлю обновление для сей статьи - сегодня-завтра собираюсь закончить, благо появилось свободное время. Не против, если использую часть твоего текста?

UPD: Буквально пару минут назад внесён коммит, добавляющий константы __file, __date и __time.

Цитата:

---

Сообщение от Daniel_Cortez

Не против, если использую часть твоего текста?

---

Конечно не против.

Цитата:

---

Сообщение от Daniel_Cortez

UPD: Буквально пару минут назад внесён коммит, добавляющий константы __file, __date и __time.

---

О, это довольно полезно.

Обновил статью. Знаю, что обещал ещё месяц назад; личные обстоятельства внесли свои коррективы.
По поводу самой статьи - вполне возможно, что упустил что-нибудь, особенно на фоне того, что у меня сейчас 2 часа ночи. Дайте знать, если заметили какой-нибудь недочёт или ошибку.

Цитата:

---

Добавлена возможность выводить пользовательские ошибки с помощью директивы #error.

---

Так это же, вроде, и в "дэфолтном" компиляторе есть, не?

Цитата:

---

recursion detected: function 40000004!=40000004 directly calls itself
recursion detected: function 40000004/0 directly calls itself
recursion detected: function 40000004*0 indirectly calls itself:
40000004*0 <- AntySH <- 40000004*0
recursion detected: function -40000004 indirectly calls itself:
-40000004 <- GetCoordBonnetVehicleEx <- -40000004
recursion detected: function AddOshive indirectly calls itself:
AddOshive <- UpdDBM <- AddOshive
...

---

И как понять где рекурсия?)
p.s. и ещё, функции AddOshive и UpdDBM ОБСАЛЮТНО не связаны... почему на них может ругаться?

Забавно...
Должно быть что-то типа такого:

  Открыть/закрыть

http://i.imgur.com/3EQORLH.png

Цитата:

---

Падение компилятора при использовании #emit sysreq.c, если указанная функция не использовалась ранее.

---

Она до сих по не исправлена (я тестировал версию из мастер-ветки), поэтому буквально десять дней назад я отправил PR с исправлением, который до сих пор не приняли.

Цитата:

---

Сообщение от Alpano

И как понять где рекурсия?)
p.s. и ещё, функции AddOshive и UpdDBM ОБСАЛЮТНО не связаны... почему на них может ругаться?

---

Покажи код.

Цитата:

---

Сообщение от DeimoS

Так это же, вроде, и в "дэфолтном" компиляторе есть, не?

---

Да, действительно, в стоковом есть директива #assert. Я добавил в статью небольшое замечание об отличии #assert от #error.

Цитата:

---

Сообщение от VVWVV

Она до сих по не исправлена (я тестировал версию из мастер-ветки), поэтому буквально десять дней назад я отправил PR с исправлением, который до сих пор не приняли.

---

Странно, я почему-то думал, что его приняли - спутал с каким-то другим, видимо. Добавил примечание в статье.

Цитата:

---

Сообщение от Daniel_Cortez

Да, действительно, в стоковом есть директива #assert. Я добавил в статью небольшое замечание об отличии #assert от #error.

---

Не, там именно директива "#error" присутствует

  Открыть/закрыть

http://i.imgur.com/boSnOTX.png

Цитата:

---

Сообщение от ziggi

Покажи код.

---

 AddOshive

PHP код:

---

stock AddOshive(playerid,ochiveid){
    if(Ochives[playerid][ochiveid]) return false;
    new points = GetOchivePoints(ochiveid),
        string[128],year, month, day;
    Player[playerid][OchivePoints] += points;
    GivePlayerCash(playerid,points*10000);
    Ochives[playerid][ochiveid] = true;
    Ochive[playerid]++;
    getdate(year, month, day);
    format(string, sizeof(string),"INSERT INTO `Ochives` VALUES ( NULL,'%d','%d','%02d.%02d.%d')",Player[playerid][pSQLID],ochiveid,day,month,year);
    db_free_result(db_query(users_base,string));
    SavePlayerInt(playerid,"OchivePoints",Player[playerid][OchivePoints]);
    if(!(playerFlags[playerid] & pMooveMode))
        GameTextForPlayer(playerid, "~w~New achieve!", 3000, 1);
    return true;
} 


---

 UpdDBM

PHP код:

---

stock UpdDBM(){
    stateDBM--;
    if(stateDBM < 0){
        stateDBM = 480+INTERVAL_DMB;
        prizeDBM = 50000;
    }
    if(stateDBM < 300+INTERVAL_DMB)
        return true;
    else if(stateDBM > 420+INTERVAL_DMB){
        new msg[64];
        format(msg,sizeof(msg),"~n~~n~~n~~n~~n~~n~~n~~n~~n~~w~Wait %d sec.",stateDBM-(420+INTERVAL_DMB));
        foreach(i){
            if(!IsPlayerInDynamicArea(i, areaDBM)) continue;
            GameTextForPlayer(i,msg,1250,3);
        }
    }else if(stateDBM == 420+INTERVAL_DMB){
        for(new i; i < 8; i++)
            SetVehicleParamsEx(dbmPOS[i][IDo],1,(hour < 7 || hour > 21)?1:0,0,0,0,0,0);
        new playersindt;
        foreach(i){
            if(!IsPlayerInDynamicArea(i, areaDBM)) continue;
            SetCameraBehindPlayer(i);
            TogglePlayerControllable(i, true);
            GameTextForPlayer(i,"~r~GO!",1500,6);
            playersindt++;
        }
        if(playersindt < 2){
            if(playersindt){
                foreach(i){
                    if(!IsPlayerInDynamicArea(i, areaDBM)) continue;
                    GivePlayerCash(i,50000);
                    break;
                }
                DellAllFromDBM();
            }
            stateDBM = 300+INTERVAL_DMB;
            return true;
        }
    }else if(stateDBM == 390+INTERVAL_DMB || stateDBM == 360+INTERVAL_DMB || stateDBM == 330+INTERVAL_DMB){
        UpdateDBM();
        foreach(i){
            if(!IsPlayerInDynamicArea(i, areaDBM)) continue;  
            GameTextForPlayer(i,"~r~LIFT DOWN!",1500,6);
        }
    }else if(stateDBM == 300+INTERVAL_DMB){
        foreach(i){
            if(!IsPlayerInDynamicArea(i, areaDBM)) continue;  
            GameTextForPlayer(i,"~g~Winner!",1000,6);
            GivePlayerCash(i,prizeDBM/3);
            GivePlayerScore(i,GetPrizeRaceExp(Player[i][pLevel]));
            AddOshive(i,WIN_DERBY);
            AddPlayerHistory(i,2,1);
        }
        DellAllFromDBM();
        st_etaj[0] = false;
        st_etaj[1] = false;
        st_etaj[2] = false;
        for(new f; f < sizeof(etaj_tre); f++)
            MoveDynamicObject(etaj_tre[f][IDo],etaj_tre[f][pPosX], etaj_tre[f][pPosY],2.9,0.3);
        for(new f; f < sizeof(etaj_two); f++)
            MoveDynamicObject(etaj_two[f][IDo],etaj_two[f][pPosX], etaj_two[f][pPosY],2.9,0.3);
        MoveDynamicObject(etaj_one[1][IDo],etaj_one[1][pPosX], etaj_one[1][pPosY],2.9,0.3);
        MoveDynamicObject(etaj_one[2][IDo],etaj_one[2][pPosX], etaj_one[2][pPosY],2.9,0.3);
    }
    return true;
} 


---

Да, во втором есть AddOshive, но обратной связи нет обсалютно.

Цитата:

---

Сообщение от Daniel_Cortez

Странно, я почему-то думал, что его приняли - спутал с каким-то другим, видимо. Добавил примечание в статье.

---

Следует обновить статью, поскольку фикс был принят.

Кроме того, теперь можно обновить библиотеки, в которых есть обход данной ошибки, чтобы не генерировать бесполезный байт-код.

Цитата:

---

Сообщение от Daniel_Cortez

Условная компиляция (#if/#elif/#else/#endif) не работала с директивой #emit.

---

Скорее всего это опечатка, ведь в Pawn нет препроцессорной директивы '#elif', ибо вместо неё есть '#elseif'.

Цитата:

---

Сообщение от Daniel_Cortez

__file - содержит строку с названием текущего исходного файла (например, "main.pwn" или "my_include.inc");

---

Следует заметить, что в инклюдах генерируется полный путь к файлу, а в исходных файлах только лишь название этого файла.

Цитата:

---

Сообщение от VVWVV

Следует обновить статью, поскольку фикс был принят.

Кроме того, теперь можно обновить библиотеки, в которых есть обход данной ошибки, чтобы не генерировать бесполезный байт-код.

---

Можно было бы определить наличие фикса по значению в константе __PawnBuild, но его так и не повысили.

Цитата:

---

Сообщение от VVWVV

Скорее всего это опечатка, ведь в Pawn нет препроцессорной директивы '#elif', ибо вместо неё есть '#elseif'.

---

Да, действительно. Исправил.

Цитата:

---

Сообщение от VVWVV

Следует заметить, что в инклюдах генерируется полный путь к файлу, а в исходных файлах только лишь название этого файла.

---

Не совсем. Допустим, что сервер находится в папке "C:\MyServer". Для исходного файла выдаётся полное имя.
Пример:

Код:

---

C:\MyServer\gamemodes\main.pwn

---

Для инклуда, подключенного с указанием относительного пути, выдаёт относительный путь:

PHP код:

---

#include "../include/test.inc" 


---

Код:

---

../include/test.inc

---

Для остальных способов подключения в __file тоже полное имя:

PHP код:

---

#include <test>
#include "test.inc"
#include <test.inc>
#include "test"
#include test 


---

Код:

---

C:\MyServer\pawno\include\test.inc

---

Похоже, компилятор не правильно компилирует конструкцию вида -

PHP код:

---

if(clickedid == TD_join[3])
    if(--model_number < 0) model_number = 7;
else if(clickedid == TD_join[4])
    if(++model_number >= 8) model_number = 0; 


---

Выдаёт:

PHP код:

---

warning 217: loose indentation 


---

Однако, если скомпилировать вот так:

PHP код:

---

if(clickedid == TD_join[3]) if(--model_number < 0) model_number = 7;
else if(clickedid == TD_join[4]) if(++model_number >= 8) model_number = 0; 


---

Ошибка пропадает, однако вторая часть конструкции:

PHP код:

---

else if(clickedid == TD_join[4]) if(++model_number >= 8) model_number = 0; 


---

Работает некорректно (точнее вовсе не работает).

Версия последняя:

PHP код:

---

Pawn compiler 3.10.2              Copyright (c) 1997-2006, ITB CompuPhase 


---

Если кто пользуется данным компилятором, проверьте проблему.

Цитата:

---

Сообщение от 123

Похоже, компилятор не правильно компилирует конструкцию вида -

PHP код:

---

if(clickedid == TD_join[3])
    if(--model_number < 0) model_number = 7;
else if(clickedid == TD_join[4])
    if(++model_number >= 8) model_number = 0; 


---

---

Компилятор всё делает правильно. Оператор else относится к последнему встреченному if, а не к первому.
Если сделать правильное выравнивание, код будет выглядеть так:

PHP код:

---

if(clickedid == TD_join[3])
    if(--model_number < 0) model_number = 7;
    else if(clickedid == TD_join[4])
        if(++model_number >= 8) model_number = 0; 


---

Или даже так:

PHP код:

---

if(clickedid == TD_join[3])
    if(--model_number < 0) model_number = 7;
    else
        if(clickedid == TD_join[4])
            if(++model_number >= 8) model_number = 0; 


---

Если же вы хотите, чтобы код в else относился к первому if, выделяйте тело ветвления фигурными скобками:

PHP код:

---

if(clickedid == TD_join[3])
{
    if(--model_number < 0) model_number = 7;
}
else if(clickedid == TD_join[4])
{
    if(++model_number >= 8) model_number = 0;
} 


---

UPD: Насколько помню, такая задача в стиле "Найди ошибку" даже есть в ЕГЭ по информатике. Или, по крайней мере, была там 5 лет тому назад.

Цитата:

---

Сообщение от Daniel_Cortez

UPD: Насколько помню, такая задача в стиле "Найди ошибку" даже есть в ЕГЭ по информатике. Или, по крайней мере, была там 5 лет тому назад.

---

И правильным ответом было "В коде всё верно! Это компилятор всё делает неправильно!!!"?

Цитата:

---

Сообщение от DeimoS

И правильным ответом было "В коде всё верно! Это компилятор всё делает неправильно!!!"?

---

Ок, возможно, я не совсем удачно всё это сформулировал. В ветках if/else было присваивание переменной разных значений и нужно было определить, какое значение в конце концов окажется в переменной. Но основная суть всё та же: неправильная табуляция могла сбить с толку.

Цитата:

---

Сообщение от Daniel_Cortez

Ок, возможно, я не совсем удачно всё это сформулировал. В ветках if/else было присваивание переменной разных значений и нужно было определить, какое значение в конце концов окажется в переменной. Но основная суть всё та же: неправильная табуляция могла сбить с толку.

---

Да не, ты всё правильно сформулировал. Это я просто решил немного устроить оффтоп и пошутить :pardon:

Вышел новый релиз 3.10.3.

Изменения:

• Добавлен новый оператор emit/__emit.
  С описанием работы данного оператора можно ознакомиться здесь.
  (Примечание (DC): В скором времени будет подготовлена русскоязычная статья по использованию оператора emit, я добавлю ссылку сюда по мере готовности.)
  
  
• Ускорена компиляция больших массивов, все элементы которых инициализированы одним и тем же значением.
  Теперь в ассемблерном листинге подобные массивы записываются как "dumpn <кол-во> <значение>".
  
  
• Теперь при возврате массивов неизвестного размера компилятор выдаёт ошибку.
  Пример:
  
    Открыть/закрыть

  
  

  PHP код:

  ---

  #include <a_samp>

GetGenderName(gender_id)
{
    static const gender_names[3][] = { {"male"}, {"female"}, {"undefined"} };
    if (0 == gender_id)
        return gender_names[0];
    if (1 == gender_id)
        return gender_names[1];
    return gender_names[2];
}

main()
{
    for (new i = 0; i < 3; ++i)
        printf("#%d: %s", i, GetGenderName(i)); // error 092: functions may not return arrays of unknown size (symbol "GetGenderName")
} 


  ---

  
  Здесь компилятор выдаёт ошибку из-за того, что не знает размер последнего измерения массива gender_names. Чтобы устранить ошибку, достаточно лишь указать размер последнего измерения массива:
  
  Код:

  ---

  static const gender_names[3][10] = { {"male"}, {"female"}, {"undefined"} };

  ---

• Устранён баг с ложным предупреждением о присвоении переменной значения из самой себя.
  Данный баг возникал при использовании операторов "-" (в унарном варианте), "!" и "~".
  
  PHP код:

  ---

  enum e_Data{
    val1, val2, val3
};
new data[e_Info];

// warning 226: a variable is assigned to itself (symbol "data")
data[val1] = -data[val1];
data[val2] = ~data[val2];
data[val3] = !data[val3]; 


  ---

• Исправлены ошибки, связанные с инициализацией многомерных массивов.
  
  
• Устранён краш при использовании #emit sysreq.c на не использовавшихся ранее в коде нативных функциях.
  (Примечание (DC): Да, опять.)
  
  
• Исправлена ошибка, из-за которой в документацию (*.xml) для автоматонов вместо имён состояний попадали мусорные данные.
  
  
• Устранён баг, из-за которого константы, указанные в аргументах командной строки, не имели никакого эффекта.
  
  
• Устранён баг, связанный с поааданием лишних пустых строк и символов '\a' в листинг препроцессора при использовании директив #pragma deprecated и #warning.
  
  
• Устранено попадание лишних пустых строк в ассемблерный листинг.
  
  
• Исправлены ошибки, возникавшие при сборке для ОС Android.

Скачать: https://github.com/Zeex/pawn/releases/tag/v3.10.3

Параллельно подготовлен тестовый выпуск, в котором, помимо всего вышеперечисленного, внесён ряд изменений для улучшения производительности.
(Примечение (DC): Особенно заметно уменьшение времени компиляции на крупных модах. Например, при компиляции мода Open-GTO на моём ПК время сборки уменьшилось с 18 до 6 секунд.)
https://github.com/Zeex/pawn/release....3-performance

Супер релиз, спасибо за участие в оптимизации скорости.

Цитата:

---

Сообщение от ziggi

Супер релиз, спасибо за участие в оптимизации скорости.

---

Да я там почти ничего не сделал, разве что немного подправил код maddinat0r'а и оптимизировал использование константы __line (последнее изначально реализовал justnonamenoname).
Другое дело вот этот патч для ещё более эффективной записи массивов с одинаковыми значениями (при сборке Open-GTO время компиляции слегка уменьшилось - с 16 до 14,5 секунд на моей машине), но я сомневаюсь, что его примут на фоне того, что сделал maddinat0r.

Мод, который компилировался за минуту с лишним, стал компилироваться за 7 секунд. Что вы там сделали такого?

Цитата:

---

Сообщение от DeimoS

Мод, который компилировался за минуту с лишним, стал компилироваться за 7 секунд. Что вы там сделали такого?

---

1) Хеш-таблицы.
2) Немного других изменений.

Вроде бы все.

Цитата:

---

Сообщение от DeimoS

Мод, который компилировался за минуту с лишним, стал компилироваться за 7 секунд. Что вы там сделали такого?

---

Львиная доля повышения производительности приходится на хранение всех глобальных идентификаторов (глобальных переменных, констант, функций) в хеш-таблице, благодаря чему для поиска того или иного идентификатора по имени не нужно просматривать весь список глобальных идентификаторов, как раньше.
Правда, цена этому - использование 128 Мб оперативы, потому что maddinat0r зачем-то посчитал нужным резервировать в хеш-таблице целых 8 миллионов слотов (хотя на самом деле их там резервируется 16 млн.), в то время как даже для самых крупных модов может хватить до 16 тысяч слотов (например, для того же Open-GTO хватило всего 4 тысяч), а даже если не хватит, таблица сама может автоматически увеличить свой размер.
Ещё немного скорость увеличилась за счёт того, что компилятор больше не выискивает встроенную константу __line в таблицах сначала локальных, а затем и глобальных идентификаторов каждый раз, когда нужно изменить её значение (а оно должно изменяться с каждой новой строкой кода). Изначально это было сделано ещё в форке justnonamenoname (также известном как "the fast russian compiler").

На самом деле, можно выжать ещё немного скорости. Не сделай maddinat0r свой Pull Request так рано, я мог бы попробовать портировать ещё пару-тройку оптимизаций из форка justnonamenoname, но теперь их уже вряд ли примут по отдельности.

Цитата:

---

Сообщение от Daniel_Cortez

Львиная доля повышения производительности приходится на хранение всех глобальных идентификаторов (глобальных переменных, констант, функций) в хеш-таблице, благодаря чему для поиска того или иного идентификатора по имени не нужно просматривать весь список глобальных идентификаторов, как раньше.
Правда, цена этому - использование 128 Мб оперативы, потому что maddinat0r зачем-то посчитал нужным резервировать в хеш-таблице целых 8 миллионов слотов (хотя на самом деле их там резервируется 16 млн.), в то время как даже для самых крупных модов может хватить до 16 тысяч слотов (например, для того же Open-GTO хватило всего 4 тысяч), а даже если не хватит, таблица сама может автоматически увеличить свой размер.
Ещё немного скорость увеличилась за счёт того, что компилятор больше не выискивает встроенную константу __line в таблицах сначала локальных, а затем и глобальных идентификаторов каждый раз, когда нужно изменить её значение (а оно должно изменяться с каждой новой строкой кода). Изначально это было сделано ещё в форке justnonamenoname (также известном как "the fast russian compiler").

На самом деле, можно выжать ещё немного скорости. Не сделай maddinat0r свой Pull Request так рано, я мог бы попробовать портировать ещё пару-тройку оптимизаций из форка justnonamenoname, но теперь их уже вряд ли примут по отдельности.

---

Другие оптимизации можно же после слияния этого PR добавить.

Для интереса попросил Eakwarp'а протестировать со своим Valakas. На сколько мне известно, размеры его amx уходят за 100 Мб.

Нашёл баг в тестовой версии. При "скреплении" строк через "\" все пробелы между двумя строками исчезают.
То бишь:

PHP код:

---

print("test  \
    test"); 


---

После компиляции пробелы перед "\" исчезнут и в консоль выдаст "testtest", а не "test test".

Цитата:

---

Сообщение от DeimoS

Нашёл баг в тестовой версии. При "скреплении" строк через "\" все пробелы между двумя строками исчезают.
То бишь:

PHP код:

---

print("test  \
    test"); 


---

После компиляции пробелы перед "\" исчезнут и в консоль выдаст "testtest", а не "test test".

---

Об этом баге уже сообщили. Остаётся надеяться, что исправят быстро и не придётся опять полгода ждать нового релиза.

Ничего себе, реально быстро компилит, раньше компилировал за 1:52 сек щас за 14 сек. :dirol::drinks:

Цитата:

---

Сообщение от ziggi

Для интереса попросил Eakwarp'а протестировать со своим Valakas. На сколько мне известно, размеры его amx уходят за 100 Мб.

---

Результаты:

Цитата:

---

3.10.2 - с итоговым AMX в 90358кб - 5 минут 30 секунд
3.10.3 - 28 секунд

---

Цитата:

---

Сообщение от ziggi

Результаты:

---

Мой amx весит 2мб, к примеру.

В целом, на версии 3.10.2 скорость компиляции была около 4-7 секунд.
На версии 3.10.3 3-6 сек.

За последние несколько дней уже успели исправить проблему с исчезающими при переносе строк пробелами, и приняли патч, в несколько раз ускоряющий процесс сборки многих крупных модов. Тем не менее, до релиза может пройти ещё неизвестно сколько времени, поэтому я подготовил "неофициальный" релиз.

Наиболее значительные изменения:

• Серия изменений для улучшения производительности компилятора (1, 2).
  
  
• Исправлена необоснованная аллокация 128 Мб памяти под хеш-таблицу.
  128 Кб (16384 слотов) вполне должно хватить даже для самых крупных модов.
  
  
• Устранён баг с исчезновением пробелов в строках.
  
  
• Исправлена сборка под Visual Studio 2010 (==> меньше жалоб от тех, у кого не установлены библиотеки VC++ Runtime старше 2010-й версии).

Скачать: Dropbox, RGHost, Яндекс.Диск
Исходный код: https://github.com/Daniel-Cortez/paw...e/experimental

Дисклеймер: Я не несу ответственности за возможный ущерб в следствие использования (в том числе неправильного использования) данного билда. Используйте на свой страх и риск. Как уже было отмечено выше, этот релиз неофициальный.

Цитата:

---

Сообщение от Daniel_Cortez

Исправлена необоснованная аллокация 128 Мб памяти под хеш-таблицу.
128 Кб (16384 слотов) вполне должно хватить даже для самых крупных модов.

---

Как я понимаю, нельзя сделать эту настройку опциональной?

Цитата:

---

Сообщение от DeimoS

Как я понимаю, нельзя сделать эту настройку опциональной?

---

На самом деле, я хотел создать issue с подобным предложением на GitHub. Но, учитывая опыт с моим предыдущим комментарием о размере хеш-таблицы (спойлер: его никто не прочёл, PR приняли без единого нарекания), я более чем уверен: это бесполезно.

Да и вообще, не думаю, что найдётся хоть один мод, который сможет превысить лимит в 16384 глобальных идентификатора.
Я пробовал скомпилировать несколько модов с патчем, позволяющим подсчитать максимально использованное количество слотов в хеш-таблице. Наибольший результат показал Open-GTO - 4120 слотов, остальные использовали не более 2 с небольшим тысяч. Даже тот же пресловутый RLS образца 2011 года заставил компилятор использовать только 1914 слотов.
Чтобы превысить изначальный размер в 16k слотов, нужен просто титанический, невиданных доселе размеров мод. Но даже если такой и существует, ничего страшного не произойдёт: хеш-таблица просто вырастет в 2 раза, до 32768 слотов.

Daniel_Cortez можешь залит на другой ФО ?

Цитата:

---

Сообщение от Blood

Daniel_Cortez можешь залит на другой ФО ?

---

Чем не устраивает dropbox? Регистрацией перед скачиванием? От неё можно отказаться, если быть хоть немного внимательным.

Цитата:

---

Сообщение от Daniel_Cortez

Чем не устраивает dropbox? Регистрацией перед скачиванием? От неё можно отказаться, если быть хоть немного внимательным.

---

Он довольно топорно работает иногда. У меня вообще минуты по 4 загружает сайт.

DC, спасибо за релиз. Скорость компиляции заметно повысилась.

// ~1.1-1.4s => ~0.4-0.6s

Цитата:

---

Сообщение от Daniel_Cortez

Чем не устраивает dropbox? Регистрацией перед скачиванием? От неё можно отказаться, если быть хоть немного внимательным.

---

При скачивание выдает ошибку .