wip 2 - new commit to track perfs on codspeed easily

Author: SuperFola

include/Ark/VM/VM.hpp

@@ -248,6 +248,15 @@ namespace Ark
          */
         [[nodiscard]] inline ARK_ALWAYS_INLINE Value* loadConstAsPtr(uint16_t id) const;
 
+        /**
+         * @brief Find the nearest variable of a given id
+         *
+         * @param id the id to find
+         * @param context
+         * @return Value*
+         */
+        inline ARK_ALWAYS_INLINE Value* findNearestVariable(uint16_t id, internal::ExecutionContext& context) noexcept;
+
         /**
          * @brief Create a new symbol with an associated value in the current scope
          *
@@ -268,9 +277,9 @@ namespace Ark
 
         inline ARK_ALWAYS_INLINE void jump(uint16_t address, internal::ExecutionContext& context);
 
-        Value getField(Value* closure, uint16_t id, const internal::ExecutionContext& context, bool push_with_env = false);
+        [[nodiscard]] Value getField(Value* closure, uint16_t id, const internal::ExecutionContext& context, bool push_with_env = false);
 
-        Value createList(std::size_t count, internal::ExecutionContext& context);
+        [[nodiscard]] Value createList(std::size_t count, internal::ExecutionContext& context);
 
         void listAppendInPlace(Value* list, std::size_t count, internal::ExecutionContext& context);
 
@@ -286,6 +295,13 @@ namespace Ark
          */
         inline ARK_ALWAYS_INLINE Value* pop(internal::ExecutionContext& context);
 
+        /**
+         * @brief Return a pointer to the top of the stack without consuming it
+         *
+         * @param context
+         * @param offset
+         * @return Value*
+         */
         inline ARK_ALWAYS_INLINE Value* peek(internal::ExecutionContext& context, std::size_t offset = 0);
 
         /**
@@ -329,19 +345,6 @@ namespace Ark
          */
         inline ARK_ALWAYS_INLINE Value* popAndResolveAsPtr(internal::ExecutionContext& context);
 
-        // ================================================
-        //                locals related
-        // ================================================
-
-        /**
-         * @brief Find the nearest variable of a given id
-         *
-         * @param id the id to find
-         * @param context
-         * @return Value*
-         */
-        inline Value* findNearestVariable(uint16_t id, internal::ExecutionContext& context) noexcept;
-
         // ================================================
         //                 function calls
         // ================================================

include/Ark/VM/VM.inl

@@ -126,6 +126,16 @@ inline ARK_ALWAYS_INLINE Value* VM::loadConstAsPtr(const uint16_t id) const
     return &m_state.m_constants[id];
 }
 
+inline ARK_ALWAYS_INLINE Value* VM::findNearestVariable(const uint16_t id, internal::ExecutionContext& context) noexcept
+{
+    for (auto it = context.locals.rbegin(), it_end = context.locals.rend(); it != it_end; ++it)
+    {
+        if (Value* val = (*it)[id]; val != nullptr)
+            return val;
+    }
+    return nullptr;
+}
+
 inline ARK_ALWAYS_INLINE void VM::store(const uint16_t id, const Value* val, internal::ExecutionContext& context)
 {
     // avoid adding the pair (id, _) multiple times, with different values
@@ -225,16 +235,6 @@ inline ARK_ALWAYS_INLINE Value* VM::popAndResolveAsPtr(internal::ExecutionContex
 
 #pragma endregion
 
-inline Value* VM::findNearestVariable(const uint16_t id, internal::ExecutionContext& context) noexcept
-{
-    for (auto it = context.locals.rbegin(), it_end = context.locals.rend(); it != it_end; ++it)
-    {
-        if (Value* val = (*it)[id]; val != nullptr)
-            return val;
-    }
-    return nullptr;
-}
-
 inline ARK_ALWAYS_INLINE void VM::returnFromFuncCall(internal::ExecutionContext& context)
 {
     --context.fc;