yaxunl added a comment.

In https://reviews.llvm.org/D31717#719316, @arsenm wrote:

> I already have https://reviews.llvm.org/D31043 which can be committed 
> independently of the alloca change


I did not know you already have a patch. Can you add reviewers to it?

The only difference is that I added AllocaInt8PtrTy to CodeGenModule and a lit 
test for non-zero alloca addr space.

I can rebase my patch on top of yours. Or I can drop this patch and add the lit 
test to your patch.


https://reviews.llvm.org/D31717



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to