JavaScript: account for the fact that our JavaScript bindings use a 32-bit address space

[agarny]

Fixes #1366.

[Copilot]

Pull request overview

Updates libCellML’s cached variable-equivalence logic to avoid cache-key collisions in 32-bit address spaces (notably the JavaScript/WASM bindings), addressing incorrect cache hits on very large models (Fixes #1366).

Changes:

• Replaces the Cantor-pairing-based cache key with a (uintptr_t, uintptr_t) key in an unordered_map for AnalyserModel::areEquivalentVariables().
• Switches generator initialisation logic to use the AnalyserModel cached areEquivalentVariables() API.
• Adds (currently disabled/commented-out) large-model regression tests across C++, Python, and JavaScript bindings.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
src/analysermodel.cpp	Uses a pointer-pair key for cached equivalence results to avoid 32-bit overflow collisions.
src/analysermodel_p.h	Introduces VariableKeyPair + custom hash and moves cache storage to unordered_map.
src/generator.cpp	Routes equivalence checks through mAnalyserModel->areEquivalentVariables() (cached).
tests/generator/generator.cpp	Adds a large-model regression test, but it is commented out.
tests/bindings/python/test_generator.py	Adds a large-model regression test, but it is disabled via a triple-quoted block.
tests/bindings/javascript/generator.test.js	Adds a large-model regression test, but it is commented out; also introduces unused imports.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.