[PATCH] D24038: Add ISD::EH_DWARF_CFA, simplify @llvm.eh.dwarf.cfa on Mips, fix on PowerPC

Simon Dardis via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 03:17:12 PDT 2016


sdardis accepted this revision.
sdardis added a comment.
This revision is now accepted and ready to land.

LGTM. Minor formatting nit inlined.


================
Comment at: lib/Target/PowerPC/PPCISelLowering.cpp:6200
@@ +6199,3 @@
+
+  int FI = MF.getFrameInfo().CreateFixedObject(isPPC64? 8 : 4, 0, false);
+  return DAG.getFrameIndex(FI, PtrVT);
----------------
Nit: space between "isPP64" and the ?.


https://reviews.llvm.org/D24038





More information about the llvm-commits mailing list