Alexander_KS/SAPFOR
4
2
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

dead_code_removing (Новый проход) #21

Merged
Alexander_KS merged 6 commits from dead_code_removing into master 2024-01-12 18:45:36 +00:00
Conversation 4 Commits 6 Files Changed 13 +1149 -406
xnpster commented 2024-01-09 15:33:53 +00:00
Collaborator
Copy Link

Вынес общую часть с обходом ГПУ в отдельный код, который можно переиспользовать и переиспользовал его вместе с другой функцией из анализа живых переменных в новом проходе удаления мёртвого кода. Анализ и удаление недостижимого кода пока не сделаны.

Вынес общую часть с обходом ГПУ в отдельный код, который можно переиспользовать и переиспользовал его вместе с другой функцией из анализа живых переменных в новом проходе удаления мёртвого кода. Анализ и удаление недостижимого кода пока не сделаны.
xnpster added 3 commits 2024-01-09 15:33:53 +00:00
extract data flow analysys part from live variable analysis 3cd16c9d0f
reorganize live variable analysis to use it's functions further 5f8940f34d
add REMOVE_DEAD_CODE_AND_UNPARSE (debug) pass 14219cdfd5
xnpster commented 2024-01-09 15:35:16 +00:00
Author
Collaborator
Copy Link

Из-за изменений в анализе живых переменных он маловероятно, но может перестать работать корректно, как и анализ приватных в следствии этого, поэтому это нужно ещё протестировать

Из-за изменений в анализе живых переменных он маловероятно, но может перестать работать корректно, как и анализ приватных в следствии этого, поэтому это нужно ещё протестировать
xnpster commented 2024-01-09 15:40:11 +00:00
Author
Collaborator
Copy Link

Ещё я не совсем уверен в том, как правильно удалять SgStatement'ы. Нужно ли ещё удалять отдельно его SgExpression'ы? Нужно ли вызывать деструктор через delete? Вроде так, как написал я, не вызывает ошибок

Ещё я не совсем уверен в том, как правильно удалять SgStatement'ы. Нужно ли ещё удалять отдельно его SgExpression'ы? Нужно ли вызывать деструктор через delete? Вроде так, как написал я, не вызывает ошибок
Alexander_KS commented 2024-01-09 15:50:15 +00:00
Owner
Copy Link

@xnpster

Ещё я не совсем уверен в том, как правильно удалять SgStatement'ы. Нужно ли ещё удалять отдельно его SgExpression'ы? Нужно ли вызывать деструктор через delete? Вроде так, как написал я, не вызывает ошибок

удалять Sg* объекты не нужно, для "удаления" SgStatement есть функция извлечения и удаления этого оператора - extract/delete соответственно. Для SgExpression достаточно присвоить NULL в соответствующее поддерево или изменить связи. Удаление объектов класса может привести к не предвиденным обстоятельствам, поэтому

  1. их не нужно размножать без надобности
  2. их не нужно удалять через delete
@xnpster > Ещё я не совсем уверен в том, как правильно удалять SgStatement'ы. Нужно ли ещё удалять отдельно его SgExpression'ы? Нужно ли вызывать деструктор через delete? Вроде так, как написал я, не вызывает ошибок удалять Sg* объекты не нужно, для "удаления" SgStatement есть функция извлечения и удаления этого оператора - extract/delete соответственно. Для SgExpression достаточно присвоить NULL в соответствующее поддерево или изменить связи. Удаление объектов класса может привести к не предвиденным обстоятельствам, поэтому 1. их не нужно размножать без надобности 2. их не нужно удалять через delete
xnpster added 1 commit 2024-01-10 19:39:32 +00:00
Live variable analysis: use vectors of blocks instead of sets bd8690d54a
xnpster commented 2024-01-10 19:42:22 +00:00
Author
Collaborator
Copy Link

Да, переход на вектора в живых переменных действительно дал прям огромное ускорение, с ~300 секунд до ~120

Да, переход на вектора в живых переменных действительно дал прям огромное ускорение, с ~300 секунд до ~120
xnpster added 1 commit 2024-01-10 20:53:39 +00:00
DATA_FLOW code style improvements: move template definitions to separate files, encapsulate CNT_NOTINIT constant de6e3bbd55
xnpster added 1 commit 2024-01-12 13:11:55 +00:00
live variables analysis: rename fcall structure to LiveDeadVarsForCall class dcb42889da
Alexander_KS merged commit f2f46e5e79 into master 2024-01-12 18:45:36 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
abrosim0v Dudarenko EgorMayorov gr_gusev IgorBobrov oleg_nikitin xnpster Alexander_KS
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Alexander_KS/SAPFOR#21
Delete Branch "dead_code_removing"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?