Skip to content

Commit

Permalink
fix(nebula_common): move EIGEN_ALIGN16 attribute to be clang-compatible
Browse files Browse the repository at this point in the history
  • Loading branch information
mojomex committed Jun 3, 2024
1 parent 65e679e commit ca3994c
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -579,7 +579,7 @@ struct FilterStatusPacket

#pragma pack(pop)

struct PointARS548Detection
struct EIGEN_ALIGN16 PointARS548Detection
{
PCL_ADD_POINT4D;
float azimuth;
Expand All @@ -598,11 +598,11 @@ struct PointARS548Detection
uint16_t object_id;
uint8_t ambiguity_flag;
EIGEN_MAKE_ALIGNED_OPERATOR_NEW
} EIGEN_ALIGN16;
};

// Note we only use a subset of the data since POINT_CLOUD_REGISTER_POINT_STRUCT has a limit in the
// number of fields
struct PointARS548Object
struct EIGEN_ALIGN16 PointARS548Object
{
PCL_ADD_POINT4D;
uint32_t id;
Expand All @@ -623,7 +623,7 @@ struct PointARS548Object
float shape_width_edge_mean;
float dynamics_orientation_rate_mean;
EIGEN_MAKE_ALIGNED_OPERATOR_NEW
} EIGEN_ALIGN16;
};

} // namespace continental_ars548
} // namespace drivers
Expand Down
8 changes: 4 additions & 4 deletions nebula_common/include/nebula_common/point_types.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@ namespace nebula
{
namespace drivers
{
struct PointXYZIR
struct EIGEN_ALIGN16 PointXYZIR
{
PCL_ADD_POINT4D;
float intensity;
uint16_t ring;
EIGEN_MAKE_ALIGNED_OPERATOR_NEW
} EIGEN_ALIGN16;
};

struct PointXYZICATR
{
Expand Down Expand Up @@ -43,7 +43,7 @@ struct PointXYZIRCAEDT
std::uint32_t time_stamp;
};

struct PointXYZIRADT
struct EIGEN_ALIGN16 PointXYZIRADT
{
PCL_ADD_POINT4D;
float intensity;
Expand All @@ -53,7 +53,7 @@ struct PointXYZIRADT
uint8_t return_type;
double time_stamp;
EIGEN_MAKE_ALIGNED_OPERATOR_NEW
} EIGEN_ALIGN16;
};

using NebulaPoint = PointXYZIRCAEDT;
using NebulaPointPtr = std::shared_ptr<NebulaPoint>;
Expand Down

0 comments on commit ca3994c

Please sign in to comment.