[PATCH] D42592: [COFF] Add minimal support for /guard:cf

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 26 13:22:02 PST 2018


pcc added inline comments.


================
Comment at: lld/COFF/InputFiles.h:191
+  // not linked into the final binary when /guard:cf is set.
+  std::vector<SectionChunk *> GuardFidChunks;
+
----------------
pcc wrote:
> Can there really be more than one `.gfids$y` section? I don't see why that would be needed.
Hmm, never mind, I see that we can get multiple sections like this:
```
void a(), b();
__declspec(selectany) void* f = a;
__declspec(selectany) void* g = b;
```


https://reviews.llvm.org/D42592





More information about the llvm-commits mailing list