PR #1505 added support for TI compiler. It explicitly added a message to notify the user that static linking is forced. PR #1755 added a more generic approach to signal such forced settings and adapted the TI-specific static linking notification to this approach. However, it turned out that TI forcibly changed another setting, too: it disabled release binary stripping, but without notification. This patch fixes this by moving the setting override to a consistent place and adding a notification. PR #1505 also added some source code changes, most importantly a complex struct initialization for a variable in `ecma-objects-general.c`. However, that initialization was coded as a macro to trick the style checker. This patch gets rid of that macro and uses proper C99 struct initializer with designators. JerryScript-DCO-1.0-Signed-off-by: Akos Kiss akiss@inf.u-szeged.hu
This commit is contained in:
+3
-3
@@ -77,8 +77,10 @@ endif()
|
||||
|
||||
if(USING_TI)
|
||||
set(ENABLE_STATIC_LINK "ON")
|
||||
set(ENABLE_STRIP "OFF")
|
||||
|
||||
set(ENABLE_STATIC_LINK_MESSAGE " (FORCED BY COMPILER)")
|
||||
set(ENABLE_STRIP_MESSAGE " (FORCED BY COMPILER)")
|
||||
endif()
|
||||
|
||||
# Status messages
|
||||
@@ -206,9 +208,7 @@ endif()
|
||||
|
||||
# Strip binary
|
||||
if(ENABLE_STRIP AND NOT CMAKE_BUILD_TYPE STREQUAL "Debug")
|
||||
if (USING_GCC OR USING_CLANG)
|
||||
jerry_add_link_flags(-s)
|
||||
endif()
|
||||
jerry_add_link_flags(-s)
|
||||
endif()
|
||||
|
||||
# External compiler & linker flags
|
||||
|
||||
Reference in New Issue
Block a user