Skip to content

[Feature]: Remove Eigen memory alignment management #2815

@jorisv

Description

@jorisv

🚀 Feature

Pinocchio 4 minimal C++ version will be C++17. Eigen memory alignment management is then no more necessary.

We should:

  • Remove EIGEN_MAKE_ALIGNED_OPERATOR_NEW
  • Remove use Eigen::aligned_allocator
  • Remove internal use of pinocchio::container::aligned_vector, PINOCCHIO_ALIGNED_STD_VECTOR and PINOCCHIO_STD_VECTOR_WITH_EIGEN_ALLOCATOR
  • Keep pinocchio::container::aligned_vector alias to maintain a stable API
  • Keep PINOCCHIO_ALIGNED_STD_VECTOR and PINOCCHIO_STD_VECTOR_WITH_EIGEN_ALLOCATOR macro to maintain a stable API

Motivation

No response

Alternatives

No response

Additional context

No response

Checklist

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions