[PATCH] D70606: LLD: CET shadow stack support on Windows

Petr Penzin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 27 16:35:52 PST 2019


penzn marked 9 inline comments as done.
penzn added a comment.

Thank you!



================
Comment at: lld/COFF/Writer.cpp:175-177
+  size_t getSize() const override {
+    return sizeof(Characteristics);
+  }
----------------
ruiu wrote:
> Let's just return 4.
Just curious, why this way - easier to read?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70606/new/

https://reviews.llvm.org/D70606





More information about the llvm-commits mailing list