[COFF] Remove a truncation assertion from setRVA
authorReid Kleckner <rnk@google.com>
Fri, 14 May 2021 02:32:49 +0000 (19:32 -0700)
committerReid Kleckner <rnk@google.com>
Fri, 14 May 2021 02:37:14 +0000 (19:37 -0700)
LLD already produces a nice error message when sections exceed 4GB, and
this setRVA assertion causes LLD to crash instead of diagnosing the
error properly.

No test because we don't want slow tests that create 4GB files.

lld/COFF/Chunks.h

index 4ecb371..bdd3faa 100644 (file)
@@ -86,8 +86,8 @@ public:
   // can be stored with 32 bits.
   uint32_t getRVA() const { return rva; }
   void setRVA(uint64_t v) {
+    // This may truncate. The writer checks for overflow later.
     rva = (uint32_t)v;
-    assert(rva == v && "RVA truncated");
   }
 
   // Returns readable/writable/executable bits.