[PATCH] D51524: [ARM64] [Windows] Handle funclets

[Mandeep Singh Grang via Phabricator via llvm-commits]

mgrang added inline comments.


================
Comment at: lib/Target/AArch64/AArch64FrameLowering.cpp:910
+static bool isFuncletReturnInstr(MachineInstr &MI) {
+  switch (MI.getOpcode()) {
+  case AArch64::CATCHRET:
----------------
default should be the first case.


================
Comment at: lib/Target/AArch64/AArch64InstrInfo.cpp:1606
+    // Skip to the first instruction before the epilog.
+    const TargetInstrInfo *TII = MBB.getParent()->getSubtarget().getInstrInfo();
+    MachineBasicBlock *TargetMBB = MI.getOperand(0).getMBB();
----------------
Can you move "MBB.getParent()-" to the next line so that it's more readable?


================
Comment at: test/CodeGen/AArch64/wineh-funclets.ll:1
+; RUN: llc -o - %s -mtriple=aarch64-windows -stop-after=prologepilog | FileCheck %s
+; This test case just checks that we can get past catchret, etc, and not the
----------------
Check 80 char limits here.


Repository:
  rL LLVM

https://reviews.llvm.org/D51524