[PATCH] D132901: [LLD] Imply "longjmp" in `/guard:cf`
Alvin Wong via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 30 03:09:07 PDT 2022
alvinhochun added inline comments.
================
Comment at: lld/COFF/DriverUtils.cpp:109-110
config->guardCF |= GuardCFLevel::CF | GuardCFLevel::LongJmp;
else if (arg.equals_insensitive("ehcont"))
config->guardCF |= GuardCFLevel::CF | GuardCFLevel::EHCont;
else
----------------
Since `ehcont` implies `cf`, perhaps in this case it should also imply `longjmp`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132901/new/
https://reviews.llvm.org/D132901
More information about the llvm-commits
mailing list