[PATCH] D115690: [SystemZ] Implement orderFrameObjects().

[Ulrich Weigand via Phabricator via llvm-commits]

uweigand added a comment.

Agreed that this version looks much more reasonable.  A few remaining comments inline.



================
Comment at: llvm/lib/Target/SystemZ/SystemZFrameLowering.cpp:165
+  };
+  std::stable_sort(SortingObjects.begin(), IE, CmpD12);
+
----------------
Why two calls to `stable_sort` here?  It seems this could be done in a single call with an appropriate comparison routine.


================
Comment at: llvm/test/CodeGen/SystemZ/stackmap.ll:474
+  %metadata2 = alloca i8, i32 4, align 4
+  %metadata3 = alloca i16, i32 4, align 4
   call void (i64, i32, i8*, i32, ...) @llvm.experimental.patchpoint.void(i64 17, i32 6, i8* null, i32 0, i8* %metadata2, i16* %metadata3)
----------------
This change seems unexpected - do you understand where this is coming from?


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

https://reviews.llvm.org/D115690