[PATCH] D68151: [GlobalISel] Add an operand to memory intrinsics to denote the "tail" marker.

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 27 11:37:39 PDT 2019


paquette added a comment.

Can we also have a check for say, `__memcpy_chk`?

e.g.

  define void @test_memcpy_chk() {
    %dst = bitcast %struct.T1* @t1 to i8*
    %src = bitcast %struct.T2* @t2 to i8*
    %ret = tail call i8* @__memcpy_chk(i8* %dst, i8* %src, i64 1824, i64 1824)
    ret void
  }

This is correctly emitted as a normal call to `memcpy` by SelectionDAG, but incorrectly emitted as a tail call by GlobalISel.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68151





More information about the llvm-commits mailing list