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

[Philip Reames via llvm-commits]

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.

Philip