Alexander_KS
  • Joined on 2023-09-14
Alexander_KS commented on pull request Alexander_KS/SAPFOR#20 2023-12-20 14:46:30 +00:00
findlocaldata

есть еще isArgInOut, надо просто посмотреть доступные функции

Alexander_KS commented on pull request Alexander_KS/SAPFOR#20 2023-12-20 14:46:30 +00:00
findlocaldata
Alexander_KS commented on pull request Alexander_KS/SAPFOR#20 2023-12-20 14:46:30 +00:00
findlocaldata

и тут

Alexander_KS commented on pull request Alexander_KS/SAPFOR#20 2023-12-20 14:46:30 +00:00
findlocaldata

а зачем тут удаляется lexNext ?

Alexander_KS commented on pull request Alexander_KS/SAPFOR#20 2023-12-20 14:46:30 +00:00
findlocaldata

зачем искать эту функцию заново, если она уже ищется до вызова ? ее нужно передать просто, и не передавать вектор

Alexander_KS commented on pull request Alexander_KS/SAPFOR#20 2023-12-20 14:46:30 +00:00
findlocaldata

std:: лишнее, так как сверху есть using namespace std;

Alexander_KS commented on pull request Alexander_KS/SAPFOR#20 2023-12-20 14:46:30 +00:00
findlocaldata

этот заголовок для отладки использовлася?

Alexander_KS created pull request Alexander_KS/SAPFOR#20 2023-12-20 14:38:15 +00:00
findlocaldata
Alexander_KS pushed to master at Alexander_KS/SAPFOR 2023-12-20 14:18:05 +00:00
6ae8dac71f improved parsing
Alexander_KS pushed to master at Alexander_KS/SAPFOR 2023-12-18 18:12:33 +00:00
41fdfcf3f8 checkpoints refactored
Alexander_KS pushed to master at Alexander_KS/SAPFOR 2023-12-16 14:42:17 +00:00
1f3d9944b8 improved and fixed FDVM, added ISNERT_INCLUDES pass
110970caa6 fixed inliner, version updated
Compare 2 commits »
Alexander_KS deleted branch planner from M/VisualSapfor 2023-12-13 09:10:46 +00:00
Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

все стандартные инклуды делаются через , а не в кавычках

Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

думаю, что можно было бы использовать просто lhs() вместо operand.

Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

оператор IMPLICIT может быть только один в каждой функции?

Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

переменная Х как то не говорит о том, что это такое

Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

третий параметр не обязателен

Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

вместо этого нужно использовать printInternalError, примеры есть почти во всех файлах проекта. Вообще везде нужно добавлять проверки и хотя бы это печатать в printInternalError, если поведение отличается от ожидаемого, чтобы потом не искать, где упало.

Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

ошибочное условие, что здесь имелось в виду? CONTROL_END является оператором закрытия любого блока, в том числе и блока описания интерфейсов. Нам достаточно убедиться, что начались исполняемые операторы или выполнялась проверка на isSgExecutableStmt(state).

Alexander_KS commented on pull request Alexander_KS/SAPFOR#19 2023-12-12 08:25:49 +00:00
task-create-implicit-pass

соответственно тут прототип должен быть SgFile*