Skip to content

Commit aaf0e1b

Browse files
committed
Add CI build with compiler optimizations
1 parent 89be0fb commit aaf0e1b

File tree

3 files changed

+27
-2
lines changed

3 files changed

+27
-2
lines changed

.github/workflows/ci.yml

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -207,6 +207,32 @@ jobs:
207207
echo "::endgroup::"
208208
echo -e "${{ env.bashPass }} ${{ env.stepName }} ${{ env.bashEnd }}"
209209
210+
- env:
211+
stepName: Build checks (Enable all functionalities with compiler optimization (speed) enabled)
212+
name: ${{ env.stepName }}
213+
run: |
214+
# ${{ env.stepName }}
215+
echo -e "::group::${{ env.bashInfo }} ${{ env.stepName }} ${{ env.bashEnd }}"
216+
217+
cmake -S . -B build -DFREERTOS_PLUS_TCP_ENABLE_BUILD_CHECKS=ON -DFREERTOS_PLUS_TCP_TEST_CONFIGURATION=ENABLE_ALL -DCMAKE_C_FLAGS="-O3"
218+
cmake --build build --target freertos_plus_tcp_build_test
219+
220+
echo "::endgroup::"
221+
echo -e "${{ env.bashPass }} ${{ env.stepName }} ${{ env.bashEnd }}"
222+
223+
- env:
224+
stepName: Build checks (Enable all functionalities with compiler optimization (size) enabled)
225+
name: ${{ env.stepName }}
226+
run: |
227+
# ${{ env.stepName }}
228+
echo -e "::group::${{ env.bashInfo }} ${{ env.stepName }} ${{ env.bashEnd }}"
229+
230+
cmake -S . -B build -DFREERTOS_PLUS_TCP_ENABLE_BUILD_CHECKS=ON -DFREERTOS_PLUS_TCP_TEST_CONFIGURATION=ENABLE_ALL -DCMAKE_C_FLAGS="-Os"
231+
cmake --build build --target freertos_plus_tcp_build_test
232+
233+
echo "::endgroup::"
234+
echo -e "${{ env.bashPass }} ${{ env.stepName }} ${{ env.bashEnd }}"
235+
210236
- env:
211237
stepName: Build checks (Enable all functionalities IPv4)
212238
name: ${{ env.stepName }}

source/FreeRTOS_DNS_Cache.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -448,7 +448,7 @@
448448
/* Add or update the item. */
449449
if( strlen( pcName ) < ( size_t ) ipconfigDNS_CACHE_NAME_LENGTH )
450450
{
451-
( void ) strncpy( xDNSCache[ uxFreeEntry ].pcName, pcName, strlen( pcName ) );
451+
( void ) strncpy( xDNSCache[ uxFreeEntry ].pcName, pcName, ipconfigDNS_CACHE_NAME_LENGTH );
452452
( void ) memcpy( &( xDNSCache[ uxFreeEntry ].xAddresses[ 0 ] ), pxIP, sizeof( *pxIP ) );
453453

454454
xDNSCache[ uxFreeEntry ].ulTTL = ulTTL;

test/build-combination/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ target_compile_options(freertos_plus_tcp
112112
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wpedantic>
113113
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wconversion>
114114
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wunused-variable>
115-
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wstringop-overflow=2>
116115
$<$<COMPILE_LANG_AND_ID:C,Clang>:-Weverything>
117116

118117
# Suppressions required to build clean with clang.

0 commit comments

Comments
 (0)