[PATCH] D40980: Add /DEBUG:GHASH option to LLD to speed up COFF linking

[Zachary Turner via Phabricator via llvm-commits]

zturner added inline comments.


================
Comment at: lld/COFF/PDB.cpp:262
+    std::vector<GloballyHashedType> OwnedHashes;
+    if (auto DebugH = getDebugH(File))
+      Hashes = getHashesFromDebugH(*DebugH);
----------------
ruiu wrote:
> auto -> Optional<ArrayRef<uint8_t>>
I agree with your previous comment about changing `auto`, but this one is a bit longer, and when initialized inside the condition of an if statement, I think it's pretty idiomatic to use `auto`.  So I would actually prefer to leave this one as is, but if you feel strongly I can still change it.


https://reviews.llvm.org/D40980