change log for gcc-testing (2010-05-13)

rtems-vc at rtems.org rtems-vc at rtems.org
Thu May 13 15:10:03 UTC 2010


 *jennifer*:
2010-05-13	Jennifer Averett <Jennifer.Averett at OARcorp.com>

	* ReportsBase.cc, ReportsHtml.cc: Added correct table structure for
	Summary table.

M  1.246  rtems-coverage/ChangeLog
M   1.11  rtems-coverage/ReportsBase.cc
M   1.16  rtems-coverage/ReportsHtml.cc

diff -u gcc-testing/rtems-coverage/ChangeLog:1.245 gcc-testing/rtems-coverage/ChangeLog:1.246
--- gcc-testing/rtems-coverage/ChangeLog:1.245	Thu May 13 09:04:11 2010
+++ gcc-testing/rtems-coverage/ChangeLog	Thu May 13 09:18:07 2010
@@ -1,3 +1,8 @@
+2010-05-13	Jennifer Averett <Jennifer.Averett at OARcorp.com>
+
+	* ReportsBase.cc, ReportsHtml.cc: Added correct table structure for
+	Summary table.
+
 2010-05-13	Joel Sherrill <joel.sherrilL at OARcorp.com>
 
 	* ReportsHtml.cc: Fix table header for disabled pagination.

diff -u gcc-testing/rtems-coverage/ReportsBase.cc:1.10 gcc-testing/rtems-coverage/ReportsBase.cc:1.11
--- gcc-testing/rtems-coverage/ReportsBase.cc:1.10	Wed May 12 11:39:01 2010
+++ gcc-testing/rtems-coverage/ReportsBase.cc	Thu May 13 09:18:07 2010
@@ -438,7 +438,7 @@
     reports->WriteSizeReport(reportName.c_str() );
 
     reportName = "symbolSummary" + reports->ReportExtension();
-    reports->WriteSizeReport(reportName.c_str() );
+    reports->WriteSymbolSummaryReport(reportName.c_str() );
   }
 
   for (ritr = reportList.begin(); ritr != reportList.end(); ritr++ ) {

diff -u gcc-testing/rtems-coverage/ReportsHtml.cc:1.15 gcc-testing/rtems-coverage/ReportsHtml.cc:1.16
--- gcc-testing/rtems-coverage/ReportsHtml.cc:1.15	Thu May 13 09:04:11 2010
+++ gcc-testing/rtems-coverage/ReportsHtml.cc	Thu May 13 09:18:07 2010
@@ -222,13 +222,23 @@
     // Put header information into the file
     fprintf(
       aFile,
-      "<table class=\"covoar-table\">\n"
-      "<tbody class=\"covoar-tbody\">\n"
-      "<tr class=\"covoar-tr covoar-tr-first\">\n"
-      "<th class=\"covoar-th\">Size</th>\n"
-      "<th class=\"covoar-th\">Symbol</th>\n"
-      "<th class=\"covoar-th\">File</th>\n"
+      "<table class=\"covoar table-autosort:0 table-autofilter"
+           TABLE_HEADER_CLASS "\">\n"
+      "<thead>\n"
+      "<tr>\n"
+      "<th class=\"table-sortable:default\" align=\"center\">Symbol</th>\n"
+      "<th class=\"table-sortable:numeric\" align=\"center\">Total</br>Size</br>Bytes</th>\n"
+      "<th class=\"table-sortable:numeric\" align=\"center\">Total</br>Size</br>Instr</th>\n"
+      "<th class=\"table-sortable:default\" align=\"center\">#</br>Ranges</th>\n"
+      "<th class=\"table-sortable:numeric\" align=\"center\">Uncovered</br>Size</br>Bytes</th>\n"
+      "<th class=\"table-sortable:numeric\" align=\"center\">Uncovered</br>Size</br>Instr</th>\n"
+      "<th class=\"table-sortable:default\" align=\"center\">#</br>Branches</th>\n"
+      "<th class=\"table-sortable:default\" align=\"center\">#</br>Always</br>Taken</th>\n"
+      "<th class=\"table-sortable:default\" align=\"center\">#</br>Never</br>Taken</th>\n"
+      "<th class=\"table-sortable:default\" align=\"center\">Percent</br>Uncovered</br>Instructions</th>\n"
+      "<th class=\"table-sortable:default\" align=\"center\">Percent</br>Uncovered</br>Bytes</th>\n"
       "</tr>\n"
+      "</thead>\n"
     );
     return aFile;
   }
@@ -448,27 +458,27 @@
     // starting line
     fprintf( 
       report, 
-      "<td class=\"covoar-td\" align=\"center\"></td>\n"
+      "<td class=\"covoar-td\" align=\"center\">???</td>\n"
      );
      
     // Size in bytes
     fprintf( 
       report, 
-      "<td class=\"covoar-td\" align=\"center\"></td>\n"
+      "<td class=\"covoar-td\" align=\"center\">???</td>\n"
     );
 
     // Size in instructions
     fprintf( 
       report, 
-      "<td class=\"covoar-td\" align=\"center\"></td>\n"
+      "<td class=\"covoar-td\" align=\"center\">???</td>\n"
     ); 
 
     // See if an explanation is available
     fprintf( 
       report, 
-      "<td class=\"covoar-td\" align=\"center\">NONE</td>\n"
+      "<td class=\"covoar-td\" align=\"center\">Unknown</td>\n"
       "<td class=\"covoar-td\" align=\"center\">"
-      "<a href=\"NotReferenced.html\">Explanation</a></td>\n"
+      "<a href=\"NotReferenced.html\">No data</a></td>\n"
     );
     WriteExplationFile( "NotReferenced.html", &explanation );
 
@@ -603,25 +613,74 @@
     else
       fprintf( report, "<tr class=\"covoar-tr covoar-tr-odd\">\n");
 
-    // size
+    // symbol
     fprintf( 
       report, 
-      "<td class=\"covoar-td\" align=\"center\">%d</td>\n",
-      range->highAddress - range->lowAddress + 1
+      "<td class=\"covoar-td\" align=\"center\">%s</td>\n",     
+      symbol->first.c_str()
     );
 
-    // symbol
+    // Total Size in Bytes
     fprintf( 
       report, 
-      "<td class=\"covoar-td\" align=\"center\">%s</td>\n",     
-      symbol->first.c_str()
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
     );
 
-    // file
+    // Total Size in Instructions 
     fprintf( 
       report, 
-      "<td class=\"covoar-td\" align=\"center\">%s</td>\n",     
-      range->lowSourceLine.c_str()
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
+    );
+
+    // Total Uncovered Ranges
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">%d</td>\n",     
+      symbol->second.uncoveredRanges->set.size()
+    );
+
+    // Uncovered Size in Bytes
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
+    );
+
+    // Uncovered Size in Instructions 
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
+    );
+
+    // Total number of branches
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">%d</td>\n",     
+      symbol->second.uncoveredBranches->set.size()
+    );
+
+    // Total Always Taken
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
+    );
+
+    // Total Never Taken
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
+    );
+
+
+    // % Uncovered Instructions 
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
+    );
+
+    // % Uncovered Bytes
+    fprintf( 
+      report, 
+      "<td class=\"covoar-td\" align=\"center\">XXX</td>\n"
     );
 
     fprintf( report, "</tr>\n");
@@ -703,13 +762,17 @@
     FILE*  aFile
   )
   {
-    fprintf( aFile, "</tbody>\n" );
-    fprintf( aFile, "</table>\n" );
-    fprintf( aFile, "</pre>\n" );
-    fprintf( aFile,"</body>\n");
-    fprintf( aFile,"</html>");
+    fprintf(
+      aFile,
+      TABLE_FOOTER
+       "</tbody>\n"
+      "</table>\n" 
+      "</pre>\n" 
+      "</body>\n"
+      "</html>"
+    );
 
-    CloseFile( aFile );
+     CloseFile( aFile );
   }
 
 }


 *jennifer*:
2010-05-13	Jennifer Averett <Jennifer.Averett at OARcorp.com>

	* DesiredSymbols.cc, DesiredSymbols.h, covoar.cc: Changed Symbol

M  1.247  rtems-coverage/ChangeLog
M    1.4  rtems-coverage/DesiredSymbols.h
M   1.10  rtems-coverage/DesiredSymbols.cc
M   1.23  rtems-coverage/covoar.cc

diff -u gcc-testing/rtems-coverage/ChangeLog:1.246 gcc-testing/rtems-coverage/ChangeLog:1.247
--- gcc-testing/rtems-coverage/ChangeLog:1.246	Thu May 13 09:18:07 2010
+++ gcc-testing/rtems-coverage/ChangeLog	Thu May 13 10:01:05 2010
@@ -1,5 +1,9 @@
 2010-05-13	Jennifer Averett <Jennifer.Averett at OARcorp.com>
 
+	* DesiredSymbols.cc, DesiredSymbols.h, covoar.cc: Changed Symbol
+
+2010-05-13	Jennifer Averett <Jennifer.Averett at OARcorp.com>
+
 	* ReportsBase.cc, ReportsHtml.cc: Added correct table structure for
 	Summary table.
 

diff -u gcc-testing/rtems-coverage/DesiredSymbols.h:1.3 gcc-testing/rtems-coverage/DesiredSymbols.h:1.4
--- gcc-testing/rtems-coverage/DesiredSymbols.h:1.3	Fri May  7 13:53:58 2010
+++ gcc-testing/rtems-coverage/DesiredSymbols.h	Thu May 13 10:01:07 2010
@@ -39,7 +39,7 @@
     /*!
      *  This member contains the size of the symbol.
      */
-    uint32_t size;
+    uint32_t sizeInBytes;
 
     /*!
      *  This member contains the disassembly associated with a symbol.
@@ -75,7 +75,7 @@
      */
     SymbolInformation() :
       baseAddress( 0 ),
-      size( 0 ),
+      sizeInBytes( 0 ),
       uncoveredBranches( NULL ),
       uncoveredRanges( NULL ),
       unifiedCoverageMap( NULL )

diff -u gcc-testing/rtems-coverage/DesiredSymbols.cc:1.9 gcc-testing/rtems-coverage/DesiredSymbols.cc:1.10
--- gcc-testing/rtems-coverage/DesiredSymbols.cc:1.9	Tue May 11 16:08:57 2010
+++ gcc-testing/rtems-coverage/DesiredSymbols.cc	Thu May 13 10:01:07 2010
@@ -163,7 +163,7 @@
       sitr->second.uncoveredBranches = theBranches;
 
       // Now scan through the coverage map of this symbol.
-      endAddress = sitr->second.size - 1;
+      endAddress = sitr->second.sizeInBytes - 1;
       a = 0;
       while (a <= endAddress) {
 
@@ -265,7 +265,7 @@
     if (itr->second.unifiedCoverageMap) {
 
       // ensure that the specified size matches the existing size.
-      if (itr->second.size != size) {
+      if (itr->second.sizeInBytes != size) {
 
         fprintf(
           stderr,
@@ -301,7 +301,7 @@
           symbolName.c_str(), 0, highAddress
         );
       itr->second.unifiedCoverageMap = aCoverageMap;
-      itr->second.size = size;
+      itr->second.sizeInBytes = size;
     }
   }
 
@@ -540,7 +540,7 @@
 
     // Ensure that the source and destination coverage maps
     // are the same size.
-    dMapSize = itr->second.size;
+    dMapSize = itr->second.sizeInBytes;
     sBaseAddress = sourceCoverageMap->getLowAddress();
     sMapSize = sourceCoverageMap->getHighAddress() - sBaseAddress + 1;
     if (dMapSize != sMapSize) {

diff -u gcc-testing/rtems-coverage/covoar.cc:1.22 gcc-testing/rtems-coverage/covoar.cc:1.23
--- gcc-testing/rtems-coverage/covoar.cc:1.22	Wed May 12 14:30:55 2010
+++ gcc-testing/rtems-coverage/covoar.cc	Thu May 13 10:01:07 2010
@@ -395,7 +395,7 @@
       theCoverageMap = itr->second.unifiedCoverageMap;
       if (theCoverageMap) {
 
-        endAddress = itr->second.size - 1;
+        endAddress = itr->second.sizeInBytes - 1;
 
         for (a = 0; a <= endAddress; a++) {
           totalBytes++;



--

Generated by Deluxe Loginfo [http://www.codewiz.org/projects/index.html#loginfo] 2.122 by Bernardo Innocenti <bernie at develer.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/vc/attachments/20100513/188a45b5/attachment.html>


More information about the vc mailing list