refactor circle command linker fix "--start..."

Author: Jason2866

builder/frameworks/espidf.py

@@ -1523,7 +1523,9 @@ def get_python_exe():
 
 # Extra flags which need to be explicitly specified in LINKFLAGS section because SCons
 # cannot merge them correctly
-extra_flags = filter_args(link_args["LINKFLAGS"], ["-T", "-u"])
+extra_flags = filter_args(
+    link_args["LINKFLAGS"], ["-T", "-u", "-Wl,--start-group", "-Wl,--end-group"]
+)
 link_args["LINKFLAGS"] = sorted(list(set(link_args["LINKFLAGS"]) - set(extra_flags)))
 
 # remove the main linker script flags '-T memory.ld'
@@ -1534,19 +1536,6 @@ def get_python_exe():
 except:
     print("Warning! Couldn't find the main linker script in the CMake code model.")
 
-# remove circle linker commands
-try:
-    link_args_index = link_args["LINKFLAGS"].index("-Wl,--start-group")
-    link_args["LINKFLAGS"].pop(link_args_index)
-except:
-    pass
-
-try:
-    link_args_index = link_args["LINKFLAGS"].index("-Wl,--end-group")
-    link_args["LINKFLAGS"].pop(link_args_index)
-except:
-    pass
-
 #
 # Process project sources
 #