---

yaml
---
r: 318958
b: refs/heads/master-rebranch
c: 71ffa22
h: refs/heads/master

Author: swift-ci

[refs]

@@ -1457,4 +1457,4 @@ refs/tags/swift-DEVELOPMENT-SNAPSHOT-2019-08-02-a: ddd2b2976aa9bfde5f20fe37f6bd2
 refs/tags/swift-DEVELOPMENT-SNAPSHOT-2019-08-03-a: 171cc166f2abeb5ca2a4003700a8a78a108bd300
 refs/heads/benlangmuir-patch-1: baaebaf39d52f3bf36710d4fe40cf212e996b212
 refs/heads/i-do-redeclare: 8c4e6d5de5c1e3f0a2cedccf319df713ea22c48e
-refs/heads/master-rebranch: 4971b164a463b7449e3bf4022450f70daa4d92c5
+refs/heads/master-rebranch: 71ffa2251536a7663cad3854a298a3f4a556205c

branches/master-rebranch/include/swift/Reflection/ReflectionContext.h

@@ -322,14 +322,21 @@ class ReflectionContext
                 : llvm::StringRef(COFFSec->Name, llvm::COFF::NameSize);
         if (SectionName != Name)
           continue;
-        auto Addr = ImageStart.getAddressData() + COFFSec->PointerToRawData;
+        auto Addr = ImageStart.getAddressData() + COFFSec->VirtualAddress;
         auto Buf = this->getReader().readBytes(RemoteAddress(Addr),
                                                COFFSec->VirtualSize);
         const char *Begin = reinterpret_cast<const char *>(Buf.get());
         const char *End = Begin + COFFSec->VirtualSize;
         savedBuffers.push_back(std::move(Buf));
-        return {{Begin, End},
-                COFFSec->VirtualAddress - COFFSec->PointerToRawData};
+
+        // FIXME: This code needs to be cleaned up and updated
+        // to make it work for 32 bit platforms.
+        if (SectionName != ".sw5cptr") {
+          Begin += 8;
+          End -= 8;
+        }
+
+        return {{Begin, End}, 0};
       }
       return {{nullptr, nullptr}, 0};
     };
@@ -338,20 +345,15 @@ class ReflectionContext
         findCOFFSectionByName(".sw5cptr");
     std::pair<std::pair<const char *, const char *>, uint32_t> TypeRefMdSec =
         findCOFFSectionByName(".sw5tyrf");
-
-    // FIXME: Make use of .sw5flmd section (the section content appears to be
-    // incorrect on Windows at the moment).
-    std::pair<std::pair<const char *, const char *>, uint32_t> FieldMdSec = {
-        {nullptr, nullptr}, 0};
-    // FIXME: Make use of .sw5asty.
-    std::pair<std::pair<const char *, const char *>, uint32_t> AssocTySec = {
-        {nullptr, nullptr}, 0};
-    // FIXME: Make use of .sw5bltn.
-    std::pair<std::pair<const char *, const char *>, uint32_t> BuiltinTySec = {
-        {nullptr, nullptr}, 0};
-    // FIXME: Make use of .sw5repl.
-    std::pair<std::pair<const char *, const char *>, uint32_t> ReflStrMdSec = {
-        {nullptr, nullptr}, 0};
+    std::pair<std::pair<const char *, const char *>, uint32_t> FieldMdSec =
+        findCOFFSectionByName(".sw5flmd");
+    std::pair<std::pair<const char *, const char *>, uint32_t> AssocTySec =
+        findCOFFSectionByName(".sw5asty");
+    // FIXME: Use the section .sw5bltn instead.
+    std::pair<std::pair<const char *, const char *>, uint32_t> BuiltinTySec =
+        {{nullptr, nullptr}, 0};
+    std::pair<std::pair<const char *, const char *>, uint32_t> ReflStrMdSec =
+        findCOFFSectionByName(".sw5rfst");
 
     if (FieldMdSec.first.first == nullptr &&
         AssocTySec.first.first == nullptr &&
@@ -457,12 +459,11 @@ class ReflectionContext
         if (SecName != Name)
           continue;
         auto SecStart =
-            RemoteAddress(ImageStart.getAddressData() + Hdr->sh_offset);
+            RemoteAddress(ImageStart.getAddressData() + Hdr->sh_addr);
         auto SecSize = Hdr->sh_size;
         auto SecBuf = this->getReader().readBytes(SecStart, SecSize);
         auto SecContents = reinterpret_cast<const char *>(SecBuf.get());
-        return {{SecContents, SecContents + SecSize},
-                Hdr->sh_addr - Hdr->sh_offset};
+        return {{SecContents, SecContents + SecSize}, 0};
       }
       return {{nullptr, nullptr}, 0};
     };