Skip to content

feat(semantic): add SymbolTable::symbol_is_mutated method#7755

Merged
graphite-app[bot] merged 1 commit intomainfrom
12-09-feat_semantic_add_symboltable_symbol_is_mutated_method
Dec 10, 2024
Merged

feat(semantic): add SymbolTable::symbol_is_mutated method#7755
graphite-app[bot] merged 1 commit intomainfrom
12-09-feat_semantic_add_symboltable_symbol_is_mutated_method

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Dec 9, 2024

Add SymbolTable::symbol_is_mutated method. Returns true if the symbol is not a const and has any references with ReferenceFlags::Write.

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 9, 2024

CodSpeed Performance Report

Merging #7755 will not alter performance

Comparing 12-09-feat_semantic_add_symboltable_symbol_is_mutated_method (78dff7d) with main (3c1b2bf)

Summary

✅ 29 untouched benchmarks

@overlookmotel overlookmotel marked this pull request as ready for review December 9, 2024 22:03
@graphite-app graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Dec 10, 2024
@graphite-app
Copy link
Contributor

graphite-app bot commented Dec 10, 2024

Merge activity

Add `SymbolTable::symbol_is_mutated` method. Returns `true` if the symbol is not a `const` and has any references with `ReferenceFlags::Write`.
@Dunqing Dunqing force-pushed the 12-09-refactor_transformer_introduce_transformctx_duplicate_expression_ branch from b23adea to b500f55 Compare December 10, 2024 02:28
@Dunqing Dunqing force-pushed the 12-09-feat_semantic_add_symboltable_symbol_is_mutated_method branch from 99542f4 to 78dff7d Compare December 10, 2024 02:29
Base automatically changed from 12-09-refactor_transformer_introduce_transformctx_duplicate_expression_ to main December 10, 2024 02:34
@graphite-app graphite-app bot merged commit 78dff7d into main Dec 10, 2024
@graphite-app graphite-app bot deleted the 12-09-feat_semantic_add_symboltable_symbol_is_mutated_method branch December 10, 2024 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0-merge Merge with Graphite Merge Queue A-semantic Area - Semantic C-enhancement Category - New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant