Changeset 2f11c36 in rtems-tools

Timestamp:

10/15/15 10:06:05 (7 years ago)

Author:

Chris Johns <chrisj@…>

Branches:

4.10, 5, master

Children:

0382b68

Parents:

f11d18b

Message:

Always provide the argument size defines.

File:

• linkers/rtems-tld.cpp (modified) (2 diffs)

linkers/rtems-tld.cpp

@@ -1235 +1235 @@
             c.write_line("");
 
-            if (sig.has_args () || (sig.has_ret () && !generator_.ret_trace.empty ()))
+            std::string ds;
+            std::string des;
+            std::string drs;
+            bool        ds_added = false;
+            bool        des_added = false;
+            bool        drs_added = false;
+            ds  = "#define FUNC_DATA_SIZE_" + sig.name + " (";
+            des = "#define FUNC_DATA_ENTRY_SIZE_" + sig.name + " (";
+            drs = "#define FUNC_DATA_RET_SIZE_" + sig.name + " (";
+
+            if (sig.has_args ())
             {
-              std::string ds;
-              std::string des;
-              std::string drs;
-              bool        ds_added = false;
-              bool        des_added = false;
-              bool        drs_added = false;
-              ds  = "#define FUNC_DATA_SIZE_" + sig.name + " (";
-              des = "#define FUNC_DATA_ENTRY_SIZE_" + sig.name + " (";
-              drs = "#define FUNC_DATA_RET_SIZE_" + sig.name + " (";
-              if (sig.has_args ())
-              {
-                for (size_t a = 0; a < sig.args.size (); ++a)
-                {
-                  if (ds_added)
-                    ds += " + ";
-                  else
-                    ds_added = true;
-                  if (des_added)
-                    des += " + ";
-                  else
-                    des_added = true;
-                  ds += "sizeof(" + sig.args[a] + ')';
-                  des += "sizeof(" + sig.args[a] + ')';
-                }
-              }
-              if (sig.has_ret () && !generator_.ret_trace.empty ())
+              for (size_t a = 0; a < sig.args.size (); ++a)
               {
                 if (ds_added)
@@ -1268 +1253 @@
                 else
                   ds_added = true;
-                if (drs_added)
-                  drs += " + ";
+                if (des_added)
+                  des += " + ";
                 else
-                  drs_added = true;
-                ds += "sizeof(" + sig.ret + ')';
-                drs += "sizeof(" + sig.ret + ')';
+                  des_added = true;
+                ds += "sizeof(" + sig.args[a] + ')';
+                des += "sizeof(" + sig.args[a] + ')';
               }
-              if (!ds_added)
-                ds += '0';
-              ds += ')';
-              if (!des_added)
-                des += '0';
-              des += ')';
-              if (!drs_added)
-                drs += '0';
-              drs += ')';
-              c.write_line(ds);
-              c.write_line(des);
-              c.write_line(drs);
             }
+            if (sig.has_ret () && !generator_.ret_trace.empty ())
+            {
+              if (ds_added)
+                ds += " + ";
+              else
+                ds_added = true;
+              if (drs_added)
+                drs += " + ";
+              else
+                drs_added = true;
+              ds += "sizeof(" + sig.ret + ')';
+              drs += "sizeof(" + sig.ret + ')';
+            }
+            if (!ds_added)
+              ds += '0';
+            ds += ')';
+            if (!des_added)
+              des += '0';
+            des += ')';
+            if (!drs_added)
+              drs += '0';
+            drs += ')';
+            c.write_line(ds);
+            c.write_line(des);
+            c.write_line(drs);
 
             c.write_line(sig.decl () + ";");