[PATCH] D37407: WholeProgramDevirt: Add import/export support for targets without absolute symbol constants.

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: llvm/lib/LTO/LTO.cpp:233
         AddUint64(ByArg.second.Info);
+        AddUnsigned(ByArg.second.Byte);
+        AddUnsigned(ByArg.second.Bit);
----------------
eugenis wrote:
> does this increase the summary size on x86, too?
Note that this is just for computation of the hash. If we were to serialize these out (see the FIXME in ModuleSummaryIndex.h) then I expect that we could arrange to write them out at the end of the record only if they are non-zero.


https://reviews.llvm.org/D37407