This patch add the missing operands to the acc.loop operation. Only the device_type
information is not part of the operation for now.
This seems unrelated?
nit: UnitAttr is already an Optional construct, so this is redundant.
nit: Remove this extra newline.
nit: Nest this within the existing if for gangNum
nit: You don't need the == ... here.
Seq is not really part of exec mapping but in group of attributes that includes auto, independent and seq and as they are mutually exclusive it doesn't really belong here anymore.
Those are UnitAttr so if they are present it means that are "active".