Usha Gupta

I presume this is fixed. We don't have any tests failing with the "AssignmentStatement" error. I also see the function `ConvertBinaryOperationTo2Params::postorder(IR::AssignmentStatement *a)` does not handle Relational operations https://github.com/p4lang/p4c/blob/main/backends/dpdk/dpdkArch.cpp#L1037

This is a bug. ckadd instruction doesn't accept metadata fields as operand. >After this modification, the calculated checksum is also incorrect, and 0x10EF needs to be added. Though ,I am...

@jafingerhut Thanks for pointing to the example programs using metadata as input to checksum extern methods. @yiding-zhou We will try to fix by next release. However, I would prefer to...

Yes @kamleshbhalui is working.

@yiding-zhou Please check if this issue is resolved.

Sorry for delayed reply. > But, the calculated checksum is also incorrect, and 0x10EF needs to be added. Perhaps, this needs to be added in the P4 program, not by...

> @usha1830 I do not know where this number 0x10EF comes from. Is it part of the packet header contents over which the checksum is being calculated? Something else? @kevin-intel...

@yiding-zhou @kevin-intel Further analysis of this issue uncovered an issue in the checksum calculation. The argument to ckadd instruction should be a multiple of 16-bit. Thanks @jafingerhut and @kamalakannanr89 for...

To confirm that the originalName is changed, I had simply added new visitors in some of the frontend passes for P4Action and ActionListElement printing the originalName. To give some context,...

For this example: https://github.com/p4lang/p4c/blob/main/testdata/p4_16_samples/pna-example-tunnel.p4 The actions that have modified originalName are decap_outer_ipv4 -> tunnel_decap_decap_outer_ipv4 set_tunnel -> tunnel_encap_set_tunnel Pasting the code from the example here for quick reference ``` * Tunnel...