[PATCH] D59216: [LLD][ELF][ARM] Redesign of .ARM.exidx handling to use a SyntheticSection
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 20 16:43:16 PDT 2019
ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: ELF/SyntheticSections.cpp:3121
+ // nullptr then it will be a synthesized EXIDX_CANTUNWIND entry.
+ ExidxEntry PrevEntry = {ulittle32_t(0), ulittle32_t(1)};
+ if (Prev)
----------------
ruiu wrote:
> Do you need to cast to ulittle32_t?
I tried this and indeed you need a cast to ulittle32_t.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59216/new/
https://reviews.llvm.org/D59216
More information about the llvm-commits
mailing list