[PATCH] D83479: [COFF] Error on unexpected .pdata size
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 9 08:55:25 PDT 2020
MaskRay added inline comments.
================
Comment at: lld/COFF/Writer.cpp:1868
+ if ((end - begin) % sizeof(Entry) != 0) {
+ fatal("unexpected .pdata size: " + std::to_string(end - begin) +
+ " is not a multiple of " + std::to_string(sizeof(Entry)));
----------------
Nit: `Twine(end-begin)`
... to avoid a temporary std::string.
================
Comment at: lld/test/COFF/pdata-arm64-bad.yaml:1
+# RUN: yaml2obj < %s > %t.obj
+#
----------------
`yaml2obj %s -o %t.obj`
Drop the `#` on the empty line.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83479/new/
https://reviews.llvm.org/D83479
More information about the llvm-commits
mailing list