[PATCH] D132901: [LLD] Imply "longjmp" in `/guard:cf`

Wed Sep 7 23:55:42 PDT 2022

mstorsjo added a comment.

In D132901#3760680 <https://reviews.llvm.org/D132901#3760680>, @pengfei wrote:

> Great job! Thanks @alvinhochun !
>
> If we consider the `longjmp` not enabling long jump is a bug, and fix the table accordingly, then it is exactly what LLD is doing with this patch. It looks to me MSVC handles the options in order too. Anyway, it matches my understanding in options parsing.

@alvinhochun Does this patch seem ok to you in this form, or do you think it would make sense to match link.exe's behaviour wrt longjmp more exactly?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132901