This is an archive of the discontinued LLVM Phabricator instance.

CodeGen: Fix TargetLowering::LowerCallTo for sret value type
ClosedPublic

Authored by yaxunl on Nov 13 2017, 8:18 PM.

Details

Summary

TargetLowering::LowerCallTo assumes that sret value type corresponds to a
pointer in default address space, which is incorrect, since sret value type
should correspond to a pointer in alloca address space, which may not
be the default address space. This causes assertion for amdgcn target
in amdgiz environment.

This patch fixes that.

Diff Detail

Repository
rL LLVM

Event Timeline

yaxunl created this revision.Nov 13 2017, 8:18 PM
arsenm accepted this revision.Nov 13 2017, 10:08 PM

LGTM

This revision is now accepted and ready to land.Nov 13 2017, 10:08 PM
This revision was automatically updated to reflect the committed changes.