[PATCH] D125645: [ARM SEH 3] [ARM] [MC] Add support for writing ARM WinEH unwind info

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

It looks like this doesn't include any directive to set the "F" bit in xdata, or the "Condition" field in epilogue scopes.  Is the omission intentional?  (I assume your LLVM patch won't emit sequences that would require either of those, but that might change, and people writing assembly might want them.)



================
Comment at: llvm/include/llvm/Support/Win64EH.h:79
+  UOP_WideSaveRegMask,
+  // Using UOP_SetFP from above
+  UOP_SaveRegsR4R7LR,
----------------
Not sure I like calling this operation UOP_SetFP, given it can set an arbitrary register.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125645