Update comment based on finding an older version of the ARM ELF specification that...
authorandrewfish <andrewfish@c2973edb-eda0-4c78-bc6a-9341b269661f>
Wed, 31 Mar 2010 18:13:20 +0000 (18:13 +0000)
committerandrewfish <andrewfish@c2973edb-eda0-4c78-bc6a-9341b269661f>
Wed, 31 Mar 2010 18:13:20 +0000 (18:13 +0000)
git-svn-id: https://edk2-buildtools.svn.sourceforge.net/svnroot/edk2-buildtools/trunk/BaseTools@1943 c2973edb-eda0-4c78-bc6a-9341b269661f

Source/C/GenFw/GenFw.c

index d6bfc5f..a8d9f52 100644 (file)
@@ -1294,10 +1294,11 @@ WriteRelocations (
         for (K = 0; K < RelSize; K += RelElementSize) {\r
 \r
           if (DynamicSegment->p_paddr == 0) {\r
         for (K = 0; K < RelSize; K += RelElementSize) {\r
 \r
           if (DynamicSegment->p_paddr == 0) {\r
-            // This seems to be how it works on armcc???? Have the email in to find out?\r
+            // Older versions of the ARM ELF (SWS ESPC 0003 B-02) specification define DT_REL\r
+            // as an offset in the dynamic segment. p_paddr is defined to be zero for ARM tools\r
             Rel = (Elf32_Rel *) ((UINT8 *) Ehdr + DynamicSegment->p_offset + RelOffset + K);\r
           } else {\r
             Rel = (Elf32_Rel *) ((UINT8 *) Ehdr + DynamicSegment->p_offset + RelOffset + K);\r
           } else {\r
-            // This is how it reads in the ELF specification\r
+            // This is how it reads in the generic ELF specification\r
             Rel = (Elf32_Rel *) ((UINT8 *) Ehdr + RelOffset + K);\r
           }\r
 \r
             Rel = (Elf32_Rel *) ((UINT8 *) Ehdr + RelOffset + K);\r
           }\r
 \r