Skip to content

[C++] Migrate to simdjson in from_string.cc and tests #50627

Closed

Description

@taepper

Describe the enhancement requested

from_string.cc should be migrated to simdjson.

Parent issue: #35460

Component(s)

C++

Activity

  1. KHARSHAVARDHAN-eng commented on Jul 26, 2026

    @KHARSHAVARDHAN-eng

    Hi!

    I'd like to work on this issue if no one is currently working on it.

    I'll investigate the existing simdjson migration pattern and submit a PR following the established implementation.

    Thanks!

  2. Reranko05 commented on Jul 26, 2026

    @Reranko05
    Collaborator

    Hi! Just a note that this issue is part of the repository-wide RapidJSON → simdjson migration (GH-35460). I am working through the migration as a series of smaller PRs to avoid duplicate work.

  3. KHARSHAVARDHAN-eng commented on Jul 26, 2026

    @KHARSHAVARDHAN-eng

    Thanks for the clarification!

    I wasn't aware this migration was already being coordinated through a series of smaller PRs.

    I've already prepared PR #50644 for from_string.cc. If this overlaps with work that's already in progress, I'm happy to close or retarget the PR.

    Otherwise, if from_string.cc is still available as part of the migration effort, I'm happy to continue updating the PR based on review feedback.

    Thanks!

  4. added a commit that references this issue on Jul 26, 2026
  5. added a commit that references this issue on Jul 28, 2026
  6. added this to the 26.0.0 milestone on Jul 28, 2026
  7. pitrou commented on Jul 28, 2026

    @pitrou
    Member

    Issue resolved by pull request 50653
    #50653

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions