carbon-language / carbon-lang

Carbon Language's main repository: documents, design, implementation, and related tools. (NOTE: Carbon Language is experimental; see README)
https://github.com/carbon-language/carbon-lang/blob/trunk/README.md
Other
32.31k stars 1.48k forks source link

Remove `inst_id` from the public interface of `ConstantId`. #4053

Closed zygoloid closed 2 weeks ago

zygoloid commented 2 weeks ago

Require mapping from a ConstantId to an InstId to go through the ConstantValueStore.

This is a preparatory step for an upcoming generics change where symbolic ConstantIds are no longer just a thin wrapper around an InstId but instead are indexes into a table with additional information about the symbolic constant beyond its InstId.