[PATCH] D132808: [LLD][MinGW] Add --[no-]guard-cf and --[no-]guard-longjmp

[Alvin Wong via Phabricator via llvm-commits]

alvinhochun added inline comments.


================
Comment at: lld/MinGW/Driver.cpp:386
+    else
+      add("-guard:cf,nolongjmp");
+  } else if (args.hasFlag(OPT_guard_longjmp, OPT_no_guard_longjmp, false)) {
----------------
mstorsjo wrote:
> If I understand correctly, the flags passed on here are unambiguous wrt the lld-link interface (and wrt D132901), so we can land this one safely now (now that D132810 is approved too)?
Yes, that's my understanding too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132808