Skip to content

Commit

Permalink
Add missing param to serialize API
Browse files Browse the repository at this point in the history
The API to serialize a runtime assumption should also take the owning
metadata, as that information would be necessary during reification if
the desire is to attach those runtime assumptions targetting a
particular compiled body with a metadata structure associated with said
compiled body.

Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
  • Loading branch information
dsouzai committed Sep 18, 2020
1 parent 25e93be commit 5b469e2
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions compiler/runtime/OMRRuntimeAssumptions.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -189,15 +189,17 @@ class RuntimeAssumption
* @brief Used to serialize an assumption to a buffer
*
* @param cursor Pointer into the buffer where the assumption to be is serialized into
* @param owningMetadata pointer to a metadata structure associated with the compiled body
* the current assumption is associated with.
*/
virtual void serialize(uint8_t *cursor) { TR_ASSERT_FATAL(false, "Should not be called\n"); }
virtual void serialize(uint8_t *cursor, uint8_t *owningMetadata) { TR_ASSERT_FATAL(false, "Should not be called\n"); }

/**
* @brief Provides the size of the serialized assumption
*
* @return Returns the size of the serialized assumption
*/
virtual uint32_t size() { TR_ASSERT_FATAL(false, "Should not be called\n"); return 0; }
virtual uint32_t size() { TR_ASSERT_FATAL(false, "Should not be called\n"); return 0; }

/*
* These functions are used to determine whether the runtime assumption falls within
Expand Down

0 comments on commit 5b469e2

Please sign in to comment.