David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 1 | // Copyright 2017 The Clspv Authors. All rights reserved. |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | // you may not use this file except in compliance with the License. |
| 5 | // You may obtain a copy of the License at |
| 6 | // |
| 7 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | // See the License for the specific language governing permissions and |
| 13 | // limitations under the License. |
| 14 | |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 15 | // Cluster POD kernel arguments. |
| 16 | // |
| 17 | // Collect plain-old-data kernel arguments and place them into a single |
| 18 | // struct argument, at the end. Other arguments are pointers, and retain |
| 19 | // their relative order. |
| 20 | // |
| 21 | // We will create a kernel function as the new entry point, and change |
| 22 | // the original kernel function into a regular SPIR function. Key |
| 23 | // kernel metadata is moved from the old function to the wrapper. |
| 24 | // We also attach a "kernel_arg_map" metadata node to the function to |
| 25 | // encode the mapping from old kernel argument to new kernel argument. |
| 26 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 27 | #include <cassert> |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 28 | #include <cstring> |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 29 | |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 30 | #include "llvm/IR/Constants.h" |
| 31 | #include "llvm/IR/DerivedTypes.h" |
| 32 | #include "llvm/IR/Function.h" |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 33 | #include "llvm/IR/IRBuilder.h" |
Diego Novillo | 3cc8d7a | 2019-04-10 13:30:34 -0400 | [diff] [blame] | 34 | #include "llvm/IR/Instructions.h" |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 35 | #include "llvm/IR/Metadata.h" |
| 36 | #include "llvm/IR/Module.h" |
| 37 | #include "llvm/Pass.h" |
| 38 | #include "llvm/Support/CommandLine.h" |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 39 | #include "llvm/Support/MathExtras.h" |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 40 | #include "llvm/Support/raw_ostream.h" |
| 41 | #include "llvm/Transforms/Utils/Cloning.h" |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 42 | |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 43 | #include "clspv/Option.h" |
| 44 | |
David Neto | 4feb7a4 | 2017-10-06 17:29:42 -0400 | [diff] [blame] | 45 | #include "ArgKind.h" |
alan-baker | c4579bb | 2020-04-29 14:15:50 -0400 | [diff] [blame] | 46 | #include "Constants.h" |
Diego Novillo | a4c44fa | 2019-04-11 10:56:15 -0400 | [diff] [blame] | 47 | #include "Passes.h" |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 48 | |
| 49 | using namespace llvm; |
| 50 | |
| 51 | #define DEBUG_TYPE "clusterpodkernelargs" |
| 52 | |
| 53 | namespace { |
| 54 | struct ClusterPodKernelArgumentsPass : public ModulePass { |
| 55 | static char ID; |
| 56 | ClusterPodKernelArgumentsPass() : ModulePass(ID) {} |
| 57 | |
| 58 | bool runOnModule(Module &M) override; |
| 59 | }; |
David Neto | 48f56a4 | 2017-10-06 16:44:25 -0400 | [diff] [blame] | 60 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 61 | } // namespace |
| 62 | |
| 63 | char ClusterPodKernelArgumentsPass::ID = 0; |
Diego Novillo | a4c44fa | 2019-04-11 10:56:15 -0400 | [diff] [blame] | 64 | INITIALIZE_PASS(ClusterPodKernelArgumentsPass, "ClusterPodKernelArgumentsPass", |
| 65 | "Cluster POD Kernel Arguments Pass", false, false) |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 66 | |
| 67 | namespace clspv { |
| 68 | llvm::ModulePass *createClusterPodKernelArgumentsPass() { |
| 69 | return new ClusterPodKernelArgumentsPass(); |
| 70 | } |
| 71 | } // namespace clspv |
| 72 | |
| 73 | bool ClusterPodKernelArgumentsPass::runOnModule(Module &M) { |
| 74 | bool Changed = false; |
| 75 | LLVMContext &Context = M.getContext(); |
| 76 | |
| 77 | SmallVector<Function *, 8> WorkList; |
| 78 | |
| 79 | for (Function &F : M) { |
| 80 | if (F.isDeclaration() || F.getCallingConv() != CallingConv::SPIR_KERNEL) { |
| 81 | continue; |
| 82 | } |
| 83 | for (Argument &Arg : F.args()) { |
| 84 | if (!isa<PointerType>(Arg.getType())) { |
| 85 | WorkList.push_back(&F); |
| 86 | break; |
| 87 | } |
| 88 | } |
| 89 | } |
| 90 | |
Diego Novillo | 3cc8d7a | 2019-04-10 13:30:34 -0400 | [diff] [blame] | 91 | SmallVector<CallInst *, 8> CallList; |
David Neto | d5b3f98 | 2017-09-28 14:49:49 -0400 | [diff] [blame] | 92 | |
Diego Novillo | 3cc8d7a | 2019-04-10 13:30:34 -0400 | [diff] [blame] | 93 | for (Function *F : WorkList) { |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 94 | Changed = true; |
| 95 | |
alan-baker | c4579bb | 2020-04-29 14:15:50 -0400 | [diff] [blame] | 96 | auto pod_arg_impl = clspv::GetPodArgsImpl(*F); |
| 97 | auto pod_arg_kind = clspv::GetArgKindForPodArgs(*F); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 98 | // An ArgMapping describes how a kernel argument is remapped. |
| 99 | struct ArgMapping { |
| 100 | std::string name; |
| 101 | // 0-based argument index in the old kernel function. |
| 102 | unsigned old_index; |
| 103 | // 0-based argument index in the new kernel function. |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 104 | int new_index; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 105 | // Offset of the argument value within the new kernel argument. |
| 106 | // This is always zero for non-POD arguments. For a POD argument, |
| 107 | // this is the byte offset within the POD arguments struct. |
| 108 | unsigned offset; |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 109 | // Size of the argument |
| 110 | unsigned arg_size; |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame] | 111 | // Argument type. |
| 112 | clspv::ArgKind arg_kind; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 113 | }; |
| 114 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 115 | // In OpenCL, kernel arguments are either pointers or POD. A composite with |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 116 | // an element or member that is a pointer is not allowed. So we'll use POD |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 117 | // as a shorthand for non-pointer. |
| 118 | |
| 119 | SmallVector<Type *, 8> PtrArgTys; |
| 120 | SmallVector<Type *, 8> PodArgTys; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 121 | SmallVector<ArgMapping, 8> RemapInfo; |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 122 | DenseMap<Argument *, unsigned> PodIndexMap; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 123 | unsigned arg_index = 0; |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 124 | int new_index = 0; |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 125 | unsigned pod_index = 0; |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 126 | for (Argument &Arg : F->args()) { |
| 127 | Type *ArgTy = Arg.getType(); |
| 128 | if (isa<PointerType>(ArgTy)) { |
| 129 | PtrArgTys.push_back(ArgTy); |
alan-baker | c4579bb | 2020-04-29 14:15:50 -0400 | [diff] [blame] | 130 | const auto kind = clspv::GetArgKind(Arg); |
alan-baker | ff6c929 | 2020-05-04 08:32:09 -0400 | [diff] [blame^] | 131 | RemapInfo.push_back( |
| 132 | {std::string(Arg.getName()), arg_index, new_index++, 0u, 0u, kind}); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 133 | } else { |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 134 | PodIndexMap[&Arg] = pod_index++; |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 135 | PodArgTys.push_back(ArgTy); |
| 136 | } |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 137 | arg_index++; |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 138 | } |
| 139 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 140 | // Put the pointer arguments first, and then POD arguments struct last. |
David Neto | 2ded02e | 2017-10-23 15:30:59 -0400 | [diff] [blame] | 141 | // Use StructType::get so we reuse types where possible. |
| 142 | auto PodArgsStructTy = StructType::get(Context, PodArgTys); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 143 | SmallVector<Type *, 8> NewFuncParamTys(PtrArgTys); |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 144 | |
alan-baker | c4579bb | 2020-04-29 14:15:50 -0400 | [diff] [blame] | 145 | if (pod_arg_impl == clspv::PodArgImpl::kUBO && |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 146 | !clspv::Option::Std430UniformBufferLayout()) { |
| 147 | SmallVector<Type *, 16> PaddedPodArgTys; |
| 148 | const DataLayout DL(&M); |
| 149 | const auto StructLayout = DL.getStructLayout(PodArgsStructTy); |
| 150 | unsigned pod_index = 0; |
| 151 | for (auto &Arg : F->args()) { |
| 152 | auto arg_type = Arg.getType(); |
| 153 | if (arg_type->isPointerTy()) |
| 154 | continue; |
| 155 | |
| 156 | // The frontend has validated individual POD arguments. When the |
| 157 | // unified struct is constructed, pad struct and array elements as |
| 158 | // necessary to achieve a 16-byte alignment. |
| 159 | if (arg_type->isStructTy() || arg_type->isArrayTy()) { |
| 160 | auto offset = StructLayout->getElementOffset(pod_index); |
| 161 | auto aligned = alignTo(offset, 16); |
| 162 | if (offset < aligned) { |
| 163 | auto int_ty = IntegerType::get(Context, 32); |
| 164 | auto char_ty = IntegerType::get(Context, 8); |
| 165 | size_t num_ints = (aligned - offset) / 4; |
| 166 | size_t num_chars = (aligned - offset) - (num_ints * 4); |
| 167 | assert((num_chars == 0 || clspv::Option::Int8Support()) && |
| 168 | "Char in UBO struct without char support"); |
| 169 | // Fix the index for the offset of the argument. |
| 170 | // Add char padding first. |
| 171 | PodIndexMap[&Arg] += num_ints + num_chars; |
| 172 | for (size_t i = 0; i < num_chars; ++i) { |
| 173 | PaddedPodArgTys.push_back(char_ty); |
| 174 | } |
| 175 | for (size_t i = 0; i < num_ints; ++i) { |
| 176 | PaddedPodArgTys.push_back(int_ty); |
| 177 | } |
| 178 | } |
| 179 | } |
| 180 | ++pod_index; |
| 181 | PaddedPodArgTys.push_back(arg_type); |
| 182 | } |
| 183 | PodArgsStructTy = StructType::get(Context, PaddedPodArgTys); |
| 184 | } |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 185 | NewFuncParamTys.push_back(PodArgsStructTy); |
| 186 | |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 187 | // We've recorded the remapping for pointer arguments. Now record the |
| 188 | // remapping for POD arguments. |
| 189 | { |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 190 | const DataLayout DL(&M); |
| 191 | const auto StructLayout = DL.getStructLayout(PodArgsStructTy); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 192 | arg_index = 0; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 193 | for (Argument &Arg : F->args()) { |
| 194 | Type *ArgTy = Arg.getType(); |
| 195 | if (!isa<PointerType>(ArgTy)) { |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 196 | unsigned arg_size = DL.getTypeStoreSize(ArgTy); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 197 | RemapInfo.push_back( |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 198 | {std::string(Arg.getName()), arg_index, new_index, |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 199 | unsigned(StructLayout->getElementOffset(PodIndexMap[&Arg])), |
alan-baker | ff6c929 | 2020-05-04 08:32:09 -0400 | [diff] [blame^] | 200 | arg_size, pod_arg_kind}); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 201 | } |
| 202 | arg_index++; |
| 203 | } |
| 204 | } |
| 205 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 206 | FunctionType *NewFuncTy = |
| 207 | FunctionType::get(F->getReturnType(), NewFuncParamTys, false); |
| 208 | |
| 209 | // Create the new function and set key properties. |
| 210 | auto NewFunc = Function::Create(NewFuncTy, F->getLinkage()); |
| 211 | // The new function adopts the real name so that linkage to the outside |
| 212 | // world remains the same. |
| 213 | NewFunc->setName(F->getName()); |
| 214 | F->setName(NewFunc->getName().str() + ".inner"); |
| 215 | |
| 216 | NewFunc->setCallingConv(F->getCallingConv()); |
| 217 | F->setCallingConv(CallingConv::SPIR_FUNC); |
| 218 | |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 219 | // Transfer attributes that don't apply to the POD arguments |
| 220 | // to the new functions. |
| 221 | auto Attributes = F->getAttributes(); |
| 222 | SmallVector<std::pair<unsigned, AttributeSet>, 8> AttrBuildInfo; |
| 223 | |
| 224 | // Return attributes have to come first |
| 225 | if (Attributes.hasAttributes(AttributeList::ReturnIndex)) { |
| 226 | auto idx = AttributeList::ReturnIndex; |
| 227 | auto attrs = Attributes.getRetAttributes(); |
| 228 | AttrBuildInfo.push_back(std::make_pair(idx, attrs)); |
| 229 | } |
| 230 | |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame] | 231 | // Then attributes for non-POD parameters |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 232 | for (auto &rinfo : RemapInfo) { |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame] | 233 | bool argIsPod = rinfo.arg_kind == clspv::ArgKind::Pod || |
alan-baker | 9b0ec3c | 2020-04-06 14:45:34 -0400 | [diff] [blame] | 234 | rinfo.arg_kind == clspv::ArgKind::PodUBO || |
| 235 | rinfo.arg_kind == clspv::ArgKind::PodPushConstant; |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame] | 236 | if (!argIsPod && Attributes.hasParamAttrs(rinfo.old_index)) { |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 237 | auto idx = rinfo.new_index + AttributeList::FirstArgIndex; |
| 238 | auto attrs = Attributes.getParamAttributes(rinfo.old_index); |
| 239 | AttrBuildInfo.push_back(std::make_pair(idx, attrs)); |
| 240 | } |
| 241 | } |
| 242 | |
alan-baker | bccf62c | 2019-03-29 10:32:41 -0400 | [diff] [blame] | 243 | // And finally function attributes. |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 244 | if (Attributes.hasAttributes(AttributeList::FunctionIndex)) { |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 245 | auto idx = AttributeList::FunctionIndex; |
alan-baker | bccf62c | 2019-03-29 10:32:41 -0400 | [diff] [blame] | 246 | auto attrs = Attributes.getFnAttributes(); |
| 247 | AttrBuildInfo.push_back(std::make_pair(idx, attrs)); |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 248 | } |
alan-baker | bccf62c | 2019-03-29 10:32:41 -0400 | [diff] [blame] | 249 | auto newAttributes = AttributeList::get(M.getContext(), AttrBuildInfo); |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 250 | NewFunc->setAttributes(newAttributes); |
| 251 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 252 | // Move OpenCL kernel named attributes. |
| 253 | // TODO(dneto): Attributes starting with kernel_arg_* should be rewritten |
| 254 | // to reflect change in the argument shape. |
alan-baker | c4579bb | 2020-04-29 14:15:50 -0400 | [diff] [blame] | 255 | auto pod_md_name = clspv::PodArgsImplMetadataName(); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 256 | std::vector<const char *> Metadatas{ |
| 257 | "reqd_work_group_size", "kernel_arg_addr_space", |
| 258 | "kernel_arg_access_qual", "kernel_arg_type", |
alan-baker | c4579bb | 2020-04-29 14:15:50 -0400 | [diff] [blame] | 259 | "kernel_arg_base_type", "kernel_arg_type_qual", |
| 260 | pod_md_name.c_str()}; |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 261 | for (auto name : Metadatas) { |
| 262 | NewFunc->setMetadata(name, F->getMetadata(name)); |
| 263 | F->setMetadata(name, nullptr); |
| 264 | } |
| 265 | |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 266 | IRBuilder<> Builder(BasicBlock::Create(Context, "entry", NewFunc)); |
| 267 | |
| 268 | // Set kernel argument mapping metadata. |
| 269 | { |
| 270 | // Attach a metadata node named "kernel_arg_map" to the new kernel |
| 271 | // function. It is a tuple of nodes, each of which is a tuple for |
| 272 | // each argument, with members: |
| 273 | // - Argument name |
| 274 | // - Ordinal index in the original kernel function |
| 275 | // - Ordinal index in the new kernel function |
| 276 | // - Byte offset within the argument. This is always 0 for pointer |
| 277 | // arguments. For POD arguments this is the offest within the POD |
| 278 | // argument struct. |
David Neto | 48f56a4 | 2017-10-06 16:44:25 -0400 | [diff] [blame] | 279 | // - Argument type |
Diego Novillo | 3cc8d7a | 2019-04-10 13:30:34 -0400 | [diff] [blame] | 280 | LLVMContext &Context = M.getContext(); |
| 281 | SmallVector<Metadata *, 8> mappings; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 282 | for (auto &arg_mapping : RemapInfo) { |
| 283 | auto *name_md = MDString::get(Context, arg_mapping.name); |
| 284 | auto *old_index_md = |
| 285 | ConstantAsMetadata::get(Builder.getInt32(arg_mapping.old_index)); |
Diego Novillo | 3cc8d7a | 2019-04-10 13:30:34 -0400 | [diff] [blame] | 286 | auto *new_index_md = |
| 287 | ConstantAsMetadata::get(Builder.getInt32(arg_mapping.new_index)); |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 288 | auto *offset_md = |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 289 | ConstantAsMetadata::get(Builder.getInt32(arg_mapping.offset)); |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 290 | auto *arg_size_md = |
| 291 | ConstantAsMetadata::get(Builder.getInt32(arg_mapping.arg_size)); |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame] | 292 | auto argKindName = GetArgKindName(arg_mapping.arg_kind); |
| 293 | auto *argtype_md = MDString::get(Context, argKindName); |
alan-baker | ff6c929 | 2020-05-04 08:32:09 -0400 | [diff] [blame^] | 294 | auto *arg_md = |
| 295 | MDNode::get(Context, {name_md, old_index_md, new_index_md, |
| 296 | offset_md, arg_size_md, argtype_md}); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 297 | mappings.push_back(arg_md); |
| 298 | } |
| 299 | |
alan-baker | ff6c929 | 2020-05-04 08:32:09 -0400 | [diff] [blame^] | 300 | NewFunc->setMetadata(clspv::KernelArgMapMetadataName(), |
| 301 | MDNode::get(Context, mappings)); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 302 | } |
| 303 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 304 | // Insert the function after the original, to preserve ordering |
| 305 | // in the module as much as possible. |
| 306 | auto &FunctionList = M.getFunctionList(); |
| 307 | for (auto Iter = FunctionList.begin(), IterEnd = FunctionList.end(); |
| 308 | Iter != IterEnd; ++Iter) { |
| 309 | if (&*Iter == F) { |
| 310 | FunctionList.insertAfter(Iter, NewFunc); |
| 311 | break; |
| 312 | } |
| 313 | } |
| 314 | |
| 315 | // The body of the wrapper is essentially a call to the original function, |
| 316 | // but we have to unwrap the non-pointer arguments from the struct. |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 317 | |
| 318 | // Map the wrapper's arguments to the callee's arguments. |
| 319 | SmallVector<Argument *, 8> CallerArgs; |
| 320 | for (Argument &Arg : NewFunc->args()) { |
| 321 | CallerArgs.push_back(&Arg); |
| 322 | } |
| 323 | Argument *PodArg = CallerArgs.back(); |
| 324 | PodArg->setName("podargs"); |
| 325 | |
| 326 | SmallVector<Value *, 8> CalleeArgs; |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 327 | unsigned podCount = 0; |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 328 | unsigned ptrIndex = 0; |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 329 | for (Argument &Arg : F->args()) { |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 330 | if (isa<PointerType>(Arg.getType())) { |
| 331 | CalleeArgs.push_back(CallerArgs[ptrIndex++]); |
| 332 | } else { |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 333 | podCount++; |
| 334 | unsigned podIndex = PodIndexMap[&Arg]; |
| 335 | CalleeArgs.push_back(Builder.CreateExtractValue(PodArg, {podIndex})); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 336 | } |
| 337 | CalleeArgs.back()->setName(Arg.getName()); |
| 338 | } |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 339 | assert(ptrIndex + podCount == F->arg_size()); |
Kévin Petit | 98d9c33 | 2019-03-13 15:03:40 +0000 | [diff] [blame] | 340 | assert(ptrIndex == PtrArgTys.size()); |
alan-baker | 038e924 | 2019-04-19 22:14:41 -0400 | [diff] [blame] | 341 | assert(podCount != 0); |
| 342 | assert(podCount == PodArgTys.size()); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 343 | |
| 344 | auto Call = Builder.CreateCall(F, CalleeArgs); |
| 345 | Call->setCallingConv(F->getCallingConv()); |
David Neto | d5b3f98 | 2017-09-28 14:49:49 -0400 | [diff] [blame] | 346 | CallList.push_back(Call); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 347 | |
| 348 | Builder.CreateRetVoid(); |
| 349 | } |
| 350 | |
David Neto | 482550a | 2018-03-24 05:21:07 -0700 | [diff] [blame] | 351 | // Inline the inner function. It's cleaner to do this. |
| 352 | for (CallInst *C : CallList) { |
| 353 | InlineFunctionInfo info; |
alan-baker | 741fd1f | 2020-04-14 17:38:15 -0400 | [diff] [blame] | 354 | Changed |= InlineFunction(*C, info).isSuccess(); |
David Neto | d5b3f98 | 2017-09-28 14:49:49 -0400 | [diff] [blame] | 355 | } |
| 356 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 357 | return Changed; |
| 358 | } |