Skip to content

fix: remove constexpr from functions using std::vector (ASAN _GLIBCXX_DEBUG compat)#22259

Closed
AztecBot wants to merge 1 commit intomerge-train/barretenbergfrom
claudebox/fix-translator-audit-ci
Closed

fix: remove constexpr from functions using std::vector (ASAN _GLIBCXX_DEBUG compat)#22259
AztecBot wants to merge 1 commit intomerge-train/barretenbergfrom
claudebox/fix-translator-audit-ci

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

@AztecBot AztecBot commented Apr 2, 2026

Summary

  • Remove constexpr from hash_to_curve and ECCVMFixedVK::get_all() — both use std::vector which is non-literal under _GLIBCXX_DEBUG (added in chore: enable _GLIBCXX_DEBUG in debug build presets #22218), breaking the ASAN build.
  • Neither function is evaluated at compile time; generators are precomputed as hardcoded values.

Verification

  • asan-fast preset: builds clean
  • GCC syntax check: passes
  • Native build: passes

ClaudeBox log: https://claudebox.work/s/501e5c6e06a49c75?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 2, 2026
@AztecBot
Copy link
Copy Markdown
Collaborator Author

AztecBot commented Apr 8, 2026

Automatically closing this stale claudebox draft PR (no updates for 5+ days). Re-open if still needed.

@AztecBot AztecBot closed this Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant