Update basic order flow to avoid allocating unnecessary memory

src/core/lib/BasicOrderFulfiller.sol

@@ -874,35 +874,6 @@ contract BasicOrderFulfiller is OrderValidator {
                  *   `keccak256(abi.encodePacked(offerItemHashes))`
                  */
                 mstore(BasicOrder_order_offerHashes_ptr, keccak256(0, OneWord))
-
-                /*
-                 * 3. Write SpentItem to offer array in OrderFulfilled event.
-                 */
-                let eventConsiderationArrPtr :=
-                    add(
-                        OrderFulfilled_offer_length_baseOffset,
-                        shl(
-                            OneWordShift,
-                            calldataload(
-                                BasicOrder_additionalRecipients_length_cdPtr
-                            )
-                        )
-                    )
-
-                // Set a length of 1 for the offer array.
-                mstore(eventConsiderationArrPtr, 1)
-
-                // Write itemType to the SpentItem struct.
-                mstore(add(eventConsiderationArrPtr, OneWord), offeredItemType)
-
-                // Copy calldata region with (offerToken, offerIdentifier,
-                // offerAmount) from OrderParameters to (token, identifier,
-                // amount) in SpentItem struct.
-                calldatacopy(
-                    add(eventConsiderationArrPtr, AdditionalRecipient_size),
-                    BasicOrder_offerToken_cdPtr,
-                    ThreeWords
-                )
             }
         }
 
@@ -1035,6 +1006,21 @@ contract BasicOrderFulfiller is OrderValidator {
                 OrderFulfilled_consideration_body_offset
             )
 
+            // Set a length of 1 for the offer array.
+            mstore(add(eventDataPtr, 0x80), 1)
+
+            // Write itemType to the SpentItem struct.
+            mstore(add(eventDataPtr, 0xa0), offeredItemType)
+
+            // Copy calldata region with (offerToken, offerIdentifier,
+            // offerAmount) from OrderParameters to (token, identifier,
+            // amount) in SpentItem struct.
+            calldatacopy(
+                add(eventDataPtr, 0xc0),
+                BasicOrder_offerToken_cdPtr,
+                ThreeWords
+            )
+
             // Derive total data size including SpentItem and ReceivedItem data.
             // SpentItem portion is already included in the baseSize constant,
             // as there can only be one element in the array.

src/core/lib/ConsiderationEncoder.sol

@@ -12,6 +12,7 @@ import {
     BasicOrder_offerer_cdPtr,
     BasicOrder_startTime_cdPtr,
     BasicOrder_startTimeThroughZoneHash_size,
+    BasicOrder_totalOriginalAdditionalRecipients_cdPtr,
     Common_amount_offset,
     Common_identifier_offset,
     Common_token_offset,
@@ -613,14 +614,19 @@ contract ConsiderationEncoder {
                 tailOffset + BasicOrder_consideration_offset_from_offer
             );
 
-            // Retrieve the offset to the length of additional recipients.
+            // Retrieve the length of additional recipients.
             uint256 additionalRecipientsLength = CalldataPointer.wrap(
                 BasicOrder_additionalRecipients_length_cdPtr
             ).readUint256();
 
+            // Retrieve the length of additional recipients.
+            uint256 totalOriginalAdditionalRecipientsLength = CalldataPointer.wrap(
+                BasicOrder_totalOriginalAdditionalRecipients_cdPtr
+            ).readUint256();
+
             // Derive offset to event data using base offset & total recipients.
             uint256 offerDataOffset = OrderFulfilled_offer_length_baseOffset
-                + (additionalRecipientsLength << OneWordShift);
+                + (totalOriginalAdditionalRecipientsLength << OneWordShift);
 
             // Derive size of offer and consideration data.
             // 2 words (lengths) + 4 (offer data) + 5 (consideration 1) + 5 * ar