Skip to content

Conversation

@Sosutha
Copy link
Owner

@Sosutha Sosutha commented Nov 8, 2024

Signed-off-by : Sosutha Sethuramapandian sosutha.sethuramapandian@intel.com

  • Added support for 128 bit constants usage in dpdk backend
  • Updated testcases and outputs

@Sosutha Sosutha force-pushed the dpdk_128_bit branch 4 times, most recently from 6ca3177 to 49488f6 Compare November 11, 2024 07:50
Copy link

@psivanup psivanup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look fine to me

auto result = new IR::IndexedVector<IR::StatOrDecl>();
if (isLargeFieldOperand(assn->left) && assn->right->is<IR::Constant>()) {
auto leftType = assn->left->type->to<IR::Type_Bits>()->width_bits();
auto rightType = assn->right->type->to<IR::Type_Bits>()->width_bits();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

misleading variable name, can use leftSize instead.

@Sosutha Sosutha force-pushed the dpdk_128_bit branch 5 times, most recently from 5ffc0a9 to c9f778b Compare November 19, 2024 07:21
Signed-off-by : Sosutha Sethuramapandian <sosutha.sethuramapandian@intel.com>

* Added support for 128 bit constants usage in dpdk backend
* Updated testcases and outputs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants