[PATCH] D82867: Fix tail allocation for large static alloca

Josh Stone via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 30 09:12:24 PDT 2020


cuviper added inline comments.


================
Comment at: llvm/test/CodeGen/X86/stack-clash-large.ll:24-25
 ; CHECK-X86-64-NEXT:	movq	$0, (%rsp)
 ; CHECK-X86-64-NEXT:	cmpq	%r11, %rsp
 ; CHECK-X86-64-NEXT:	jl	.LBB0_1
 ; CHECK-X86-64-NEXT:# %bb.2:
----------------
I think this condition is also backwards -- in AT&T syntax, this will jump when %rsp<%r11, which is immediately false.


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

https://reviews.llvm.org/D82867





More information about the llvm-commits mailing list