[PATCH] D116474: [lld] [coff] Fix the characteristics of some sections like ".voltbl".
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jan 1 15:19:09 PST 2022
mstorsjo added inline comments.
================
Comment at: lld/COFF/Writer.cpp:1505
+ // Or the program will be crash sometimes.
+ if (sec->header.Characteristics == 0 && sec->name.contains(".voltbl")) {
+ sec->header.Characteristics |= IMAGE_SCN_CNT_INITIALIZED_DATA |
----------------
I'd prefer to have this either be `sec->name == ".voltbl"` or `sec->name.startswith(".voltbl")` instead, to make it a bit more strict still.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116474/new/
https://reviews.llvm.org/D116474
More information about the llvm-commits
mailing list