fix ci issues to setup environment

Signed-off-by: jiseong.oh <[email protected]>

Author: Jiseong-oh

.ci/scripts/setup-samsung-linux-deps.sh

@@ -30,11 +30,11 @@ download_ai_lite_core() {
 }
 
 install_enn_backend() {
-  # Please export ANDROID_NDK_ROOT if enable on-device test
+  # build Exynos backend
+  export ANDROID_NDK_ROOT=${ANDROID_NDK_ROOT:-/opt/ndk}
+  bash backends/samsung/build.sh --build all
+  # set env variable
   export EXECUTORCH_ROOT="$(cd -- "$(dirname -- "${BASH_SOURCE[0]}")/../.." && pwd)"
-
-  # Remove --build and add --ndk parameter if enable on-device test
-  bash backends/samsung/build.sh --build x86_64
   export PYTHONPATH=${PYTHONPATH:-}:${EXECUTORCH_ROOT}/..
 }
 

.github/workflows/pull.yml

@@ -873,8 +873,8 @@ jobs:
       id-token: write
       contents: read
     with:
-      runner: linux.arm64.2xlarge
-      docker-image: ci-image:executorch-ubuntu-22.04-gcc11-aarch64
+      runner: linux.2xlarge
+      docker-image: ci-image:executorch-ubuntu-22.04-gcc9
       submodules: 'recursive'
       ref: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
       timeout: 90

backends/samsung/aot/wrappers/op_param_wrapper.h

@@ -73,19 +73,19 @@ class OpParamWrapper {
 
   template <typename T>
   void SetScalarValue(T value) {
-    auto bytes_ = sizeof(T);
-    storage_ = std::unique_ptr<uint8_t[]>(new uint8_t[bytes_]);
-    memcpy(storage_.get(), &value, bytes_);
+    auto bytes = sizeof(T);
+    storage_ = std::unique_ptr<uint8_t[]>(new uint8_t[bytes]);
+    memcpy(storage_.get(), &value, bytes);
     size_ = 1;
     is_scalar_ = true;
     scalar_type_ = ScalarTypeCast<T>::value;
   }
 
   template <typename T>
   void SetVectorValue(const std::vector<T>& value) {
-    auto bytes_ = sizeof(T) * value.size();
-    storage_ = std::unique_ptr<uint8_t[]>(new uint8_t[bytes_]);
-    memcpy(storage_.get(), value.data(), bytes_);
+    auto bytes = sizeof(std::string::value_type) * value.size();
+    storage_ = std::unique_ptr<uint8_t[]>(new uint8_t[bytes]);
+    memcpy(storage_.get(), value.data(), bytes);
     size_ = value.size();
     is_scalar_ = false;
     scalar_type_ = ScalarTypeCast<T>::value;
@@ -114,7 +114,6 @@ class OpParamWrapper {
   std::string key_name_;
   std::unique_ptr<uint8_t[]> storage_ = nullptr;
   uint32_t size_ = 0;
-  uint32_t bytes_ = 0;
   bool is_scalar_ = false;
   ScalarType scalar_type_ = ScalarType::UNKNOWN;
 };