[PATCH] D88641: [AArch64] Omit SEH directives for the epilogue if none are needed

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 07:11:19 PDT 2020


mstorsjo added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64FrameLowering.cpp:1641
+    // HasEpilogStart is equal to HasWinCFI, to verify this assumption.
     HasWinCFI = true;
+    HasEpilogStart = true;
----------------
I guess an alternative, assert-less approach, would be not to set HaveWinCFI here, and remove the EpilogStart at the end, if it turns out not to be needed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88641



More information about the llvm-commits mailing list