Merge branch 'main' into permanent-hare

.github/workflows/actionlint.yml

@@ -0,0 +1,22 @@
+name: Actionlint
+
+on:
+  push:
+    branches: [ main ]
+    paths:
+      - ".github/workflows/*.ya?ml"
+  pull_request:
+    paths:
+      - ".github/workflows/*.ya?ml"
+  workflow_dispatch:
+
+jobs:
+  actionlint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Run actionlint
+        run: |
+          bash <(curl https://raw.githubusercontent.com/rhysd/actionlint/main/scripts/download-actionlint.bash)
+          ./actionlint -color

.github/workflows/build.yml

@@ -40,12 +40,14 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
+        with:
+          path: spirit
 
       - name: build-mbed-cli
         run: |
           mbed new .
           mbed deploy
-          cp .github/workflows/build-mbed/main.cpp .
+          cp spirit/.github/workflows/build-mbed/main.cpp .
           mbed compile -t GCC_ARM -m ${{ matrix.target }} --profile ${{ matrix.profile }}
 
   build-cli-v2:
@@ -61,10 +63,11 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
+        with:
+          path: spirit
 
       - name: build-mbed-tools
         run: |
-          cd ..
           mbed-tools new .
           cp spirit/.github/workflows/build-mbed/main.cpp .
           cp spirit/.github/workflows/build-mbed-tools/CMakeLists.txt .

CMakeLists.txt

@@ -1,5 +1,7 @@
 cmake_minimum_required(VERSION 3.14)
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 enable_language(CXX)
 set(CMAKE_CXX_STANDARD 14)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)

CONTRIBUTING.md

@@ -12,19 +12,23 @@
 
 反映してほしい修正を送ってください
 
-規模も小さく、開発立ち上げ直後なので、関数名の名前変更や構造自体の変更などの**破壊的な変更**も積極的に受け付けています
+規模も小さく、開発立ち上げ直後なので、関数名の名前変更や構造自体の変更などの**破壊的な変更**の提案も積極的に受け付けています
+
+Issue の作成や議論後に Pull request を出しても良いですが、いきなり Pull request を出しても構いません
 
 ## Code style
 
 基本的には、既存のコードの書き方を踏襲して書いてください
 
 いくつかのお願いを以下に記します
 
+他にもお願いをする場合がありますが、ご了承のほどよろしくお願いいたします
+
 ### 命名規則
 
 - PascalCase
-  - クラス名
-  - Enum, Enum class 名
+  - class
+  - enum, enum class
 - snake_case
   - namespace
   - function
@@ -37,3 +41,5 @@
 - private なメンバ変数は先頭に `_` をつける
 - public な関数には Doxygen 形式のコメントを付ける
 - ClangFormat を使って自動整形する
+- cspell を使ってスペルチェックする
+  - 辞書に登録されていない単語は、辞書に登録する

src/spirit/src/bfloat16.cpp

@@ -25,8 +25,10 @@ float bfloat16_to_float32(const uint16_t bfloat16)
             return std::numeric_limits<float>::quiet_NaN() * sign;
         }
     } else {
-        constexpr uint32_t exponent_bias = 127 + 7;  // 128: 指数部の0x7Fのオフセット, 7: 仮数部のビット数
-        return std::ldexp(static_cast<float>(mantissa) + 128.0F, exponent - exponent_bias) * sign;
+        constexpr uint32_t exponent_bias = 127 + 7;  // 127: 指数部の0x7Fのオフセット, 7: 仮数部のビット数
+        const uint32_t actual_mantissa = mantissa + 128;  // ケチ表現で省略された最上位ビットを復元
+
+        return std::ldexp(actual_mantissa, exponent - exponent_bias) * sign;
     }
 }