Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unify symbol reference handling #723

Open
6 of 13 tasks
xlauko opened this issue Sep 11, 2024 · 0 comments
Open
6 of 13 tasks

Unify symbol reference handling #723

xlauko opened this issue Sep 11, 2024 · 0 comments
Assignees
@xlauko
Labels
high-level

Comments

@xlauko
Copy link
Member

xlauko commented Sep 11, 2024 •
edited
Loading

Prerequisites

  • I have read the documentation and the proposed feature is not implemented.

Description

  • remove SSA value from DeclRefOp and make it take FlatSymbolRefAttr instead
  • remove obsolete graph regions
  • unify GlobalRef and DeclRef to single operation as they will behave the same now
  • rename RecordMemberOp to MemberRefOp
  • let RecordType use SymbolRefParameter instead of a pure StringRefParameter
  • let EnumType use SymbolRefParameter instead of a pure StringRefParameter
  • let TypedefType use SymbolRefParameter instead of a pure StringRefParameter
  • resurrect vast-query to test symbol API
  • remove util/Symbols.hpp in favor of generic SymbolTable lookup
  • fix VarDeclOp::getDeclContextKind to use vast symbol tables (use get_effective_symbol_table_for)
  • replace SymbolTable::getSymbolAttrName() with symbol_attr_name
  • create decl ref interface to unify all symbol referencing operations
@xlauko xlauko self-assigned this Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xlauko xlauko

Labels
high-level
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xlauko