[PATCH] D78717: [SystemZ] Implement -fstack-clash-protection

Sat Jun 6 10:08:22 PDT 2020

jonpa marked 5 inline comments as done.
jonpa added a comment.

Thanks for review - committed as 515bfc6 <https://reviews.llvm.org/rG515bfc66eaced830c03b2ec187bef0d8c4dc6915> after last updates.

================
Comment at: llvm/lib/Target/SystemZ/SystemZInstrInfo.td:38
+
+let hasNoSchedulingInfo = 1 in
+  def STACKALLOC_W_PROBING : Pseudo<(outs), (ins i64imm:$stacksize), []>;
----------------
uweigand wrote:
> Doesn't this also need Defs and Uses to be fully correct?
Ah, yes I suppose that might as well be there... I added the defs and uses and also the side-effects flag since it may expand into a loop.

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

https://reviews.llvm.org/D78717