[llvm] r263158 - Support arbitrary addrspace pointers in masked load/store intrinsics

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 16 15:29:01 PDT 2016

On 03/16/2016 03:20 PM, Manman Ren via llvm-commits wrote:
>> +  case Intrinsic::masked_store: {
>> >+    SmallVector<Value *, 4> Args(CI->arg_operands().begin(),
>> >+                                 CI->arg_operands().end());
>> >+    CI->replaceAllUsesWith(Builder.CreateCall(NewFn, Args));
> Should we keep the metadata here if it exists on the original call instruction?
Possibly, but if we do we should be consistent.  We drop metadata for 
all of the other intrinsics being upgraded too.


More information about the llvm-commits mailing list