[PATCH] D139437: [SimplifyCFG] Prohibit hoisting of llvm.deoptimize calls

[Dmitry Makogon via Phabricator via llvm-commits]

dmakogon marked an inline comment as done.
dmakogon added inline comments.


================
Comment at: llvm/test/Transforms/SimplifyCFG/dont-hoist-deoptimize.ll:31
   %tmp3 = trunc i64 0 to i32
   call void (...) @llvm.experimental.deoptimize.isVoid(i32 13) #0 [ "deopt"(i32 0, i32 1, i32 0, i32 502, i32 4, i32 35, i32 0, i32 0, ptr addrspace(1) null, i32 3, i32 -99, i32 0, ptr addrspace(1) null, i32 3, i32 -99, i32 0, ptr addrspace(1) null, i32 7, ptr null, i32 3, i32 0, i32 3, i32 0, i32 3, i32 %tmp3, i32 3, i32 0, i32 3, i32 0, i32 3, i32 %tmp, i32 3, i32 0, i32 3, i32 -99, i32 3, i32 0, i32 3, i32 14, i32 3, i32 0, i32 3, i32 -99, i32 3, i32 0, i32 3, i32 0, i32 3, i32 0, i32 3, i32 0, i32 0, ptr addrspace(1) null, i32 3, float 0.000000e+00, i32 4, double 0.000000e+00, i32 7, ptr null, i32 0, ptr addrspace(1) null, i32 0, ptr addrspace(1) null, i32 0, ptr addrspace(1) null, i32 7, ptr null, i32 7, ptr null, i32 0, ptr addrspace(1) null, i32 0, ptr addrspace(1) null, i32 0, ptr addrspace(1) null, i32 0, ptr addrspace(1) null, i32 7, ptr null, i32 7, ptr null, i32 7, ptr null, i32 7, ptr null) ]
   ret void
----------------
fhahn wrote:
> Is this massive bundle needed or can it be reduced?
> 
> Same for the bundle below.
Yeah, they may be reduced. Thanks! Landed reduced version


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

https://reviews.llvm.org/D139437