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" |
| 33 | #include "llvm/IR/Instructions.h" |
| 34 | #include "llvm/IR/IRBuilder.h" |
| 35 | #include "llvm/IR/Metadata.h" |
| 36 | #include "llvm/IR/Module.h" |
| 37 | #include "llvm/Pass.h" |
| 38 | #include "llvm/Support/CommandLine.h" |
| 39 | #include "llvm/Support/raw_ostream.h" |
| 40 | #include "llvm/Transforms/Utils/Cloning.h" |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 41 | |
David Neto | 4feb7a4 | 2017-10-06 17:29:42 -0400 | [diff] [blame] | 42 | #include "ArgKind.h" |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 43 | |
| 44 | using namespace llvm; |
| 45 | |
| 46 | #define DEBUG_TYPE "clusterpodkernelargs" |
| 47 | |
| 48 | namespace { |
| 49 | struct ClusterPodKernelArgumentsPass : public ModulePass { |
| 50 | static char ID; |
| 51 | ClusterPodKernelArgumentsPass() : ModulePass(ID) {} |
| 52 | |
| 53 | bool runOnModule(Module &M) override; |
| 54 | }; |
David Neto | 48f56a4 | 2017-10-06 16:44:25 -0400 | [diff] [blame] | 55 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 56 | } // namespace |
| 57 | |
| 58 | char ClusterPodKernelArgumentsPass::ID = 0; |
| 59 | static RegisterPass<ClusterPodKernelArgumentsPass> |
| 60 | X("ClusterPodKernelArgumentsPass", |
| 61 | "Cluster POD Kernel Arguments Pass"); |
| 62 | |
| 63 | namespace clspv { |
| 64 | llvm::ModulePass *createClusterPodKernelArgumentsPass() { |
| 65 | return new ClusterPodKernelArgumentsPass(); |
| 66 | } |
| 67 | } // namespace clspv |
| 68 | |
| 69 | bool ClusterPodKernelArgumentsPass::runOnModule(Module &M) { |
| 70 | bool Changed = false; |
| 71 | LLVMContext &Context = M.getContext(); |
| 72 | |
| 73 | SmallVector<Function *, 8> WorkList; |
| 74 | |
| 75 | for (Function &F : M) { |
| 76 | if (F.isDeclaration() || F.getCallingConv() != CallingConv::SPIR_KERNEL) { |
| 77 | continue; |
| 78 | } |
| 79 | for (Argument &Arg : F.args()) { |
| 80 | if (!isa<PointerType>(Arg.getType())) { |
| 81 | WorkList.push_back(&F); |
| 82 | break; |
| 83 | } |
| 84 | } |
| 85 | } |
| 86 | |
David Neto | d5b3f98 | 2017-09-28 14:49:49 -0400 | [diff] [blame] | 87 | SmallVector<CallInst*, 8> CallList; |
| 88 | |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 89 | // Note: The transformation done in this pass preserves the pointer-to-local |
| 90 | // arg to spec-id mapping. |
| 91 | clspv::ArgIdMapType arg_spec_id_map = clspv::AllocateArgSpecIds(M); |
| 92 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 93 | for (Function* F : WorkList) { |
| 94 | Changed = true; |
| 95 | |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 96 | // An ArgMapping describes how a kernel argument is remapped. |
| 97 | struct ArgMapping { |
| 98 | std::string name; |
| 99 | // 0-based argument index in the old kernel function. |
| 100 | unsigned old_index; |
| 101 | // 0-based argument index in the new kernel function. |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 102 | int new_index; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 103 | // Offset of the argument value within the new kernel argument. |
| 104 | // This is always zero for non-POD arguments. For a POD argument, |
| 105 | // this is the byte offset within the POD arguments struct. |
| 106 | unsigned offset; |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 107 | // Size of the argument |
| 108 | unsigned arg_size; |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame^] | 109 | // Argument type. |
| 110 | clspv::ArgKind arg_kind; |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 111 | // If non-negative, this argument is a pointer-to-local, and the value |
| 112 | // here is the specialization constant id for the array size. |
| 113 | int spec_id; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 114 | }; |
| 115 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 116 | // 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] | 117 | // 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] | 118 | // as a shorthand for non-pointer. |
| 119 | |
| 120 | SmallVector<Type *, 8> PtrArgTys; |
| 121 | SmallVector<Type *, 8> PodArgTys; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 122 | SmallVector<ArgMapping, 8> RemapInfo; |
| 123 | unsigned arg_index = 0; |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 124 | int new_index = 0; |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 125 | for (Argument &Arg : F->args()) { |
| 126 | Type *ArgTy = Arg.getType(); |
| 127 | if (isa<PointerType>(ArgTy)) { |
| 128 | PtrArgTys.push_back(ArgTy); |
David Neto | 862b7d8 | 2018-06-14 18:48:37 -0400 | [diff] [blame] | 129 | const auto kind = clspv::GetArgKindForType(ArgTy); |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 130 | int spec_id = -1; |
David Neto | 862b7d8 | 2018-06-14 18:48:37 -0400 | [diff] [blame] | 131 | if (kind == clspv::ArgKind::Local) { |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 132 | spec_id = arg_spec_id_map[&Arg]; |
| 133 | assert(spec_id > 0); |
| 134 | } |
David Neto | 862b7d8 | 2018-06-14 18:48:37 -0400 | [diff] [blame] | 135 | RemapInfo.push_back({std::string(Arg.getName()), arg_index, new_index++, |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame^] | 136 | 0u, 0u, kind, spec_id}); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 137 | } else { |
| 138 | PodArgTys.push_back(ArgTy); |
| 139 | } |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 140 | arg_index++; |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 141 | } |
| 142 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 143 | // Put the pointer arguments first, and then POD arguments struct last. |
David Neto | 2ded02e | 2017-10-23 15:30:59 -0400 | [diff] [blame] | 144 | // Use StructType::get so we reuse types where possible. |
| 145 | auto PodArgsStructTy = StructType::get(Context, PodArgTys); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 146 | SmallVector<Type *, 8> NewFuncParamTys(PtrArgTys); |
| 147 | NewFuncParamTys.push_back(PodArgsStructTy); |
| 148 | |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 149 | // We've recorded the remapping for pointer arguments. Now record the |
| 150 | // remapping for POD arguments. |
| 151 | { |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 152 | const DataLayout DL(&M); |
| 153 | const auto StructLayout = DL.getStructLayout(PodArgsStructTy); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 154 | arg_index = 0; |
| 155 | int pod_index = 0; |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 156 | for (Argument &Arg : F->args()) { |
| 157 | Type *ArgTy = Arg.getType(); |
| 158 | if (!isa<PointerType>(ArgTy)) { |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 159 | unsigned arg_size = DL.getTypeStoreSize(ArgTy); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 160 | RemapInfo.push_back( |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 161 | {std::string(Arg.getName()), arg_index, new_index, |
David Neto | 48f56a4 | 2017-10-06 16:44:25 -0400 | [diff] [blame] | 162 | unsigned(StructLayout->getElementOffset(pod_index++)), |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame^] | 163 | arg_size, clspv::GetArgKindForType(ArgTy), -1}); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 164 | } |
| 165 | arg_index++; |
| 166 | } |
| 167 | } |
| 168 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 169 | FunctionType *NewFuncTy = |
| 170 | FunctionType::get(F->getReturnType(), NewFuncParamTys, false); |
| 171 | |
| 172 | // Create the new function and set key properties. |
| 173 | auto NewFunc = Function::Create(NewFuncTy, F->getLinkage()); |
| 174 | // The new function adopts the real name so that linkage to the outside |
| 175 | // world remains the same. |
| 176 | NewFunc->setName(F->getName()); |
| 177 | F->setName(NewFunc->getName().str() + ".inner"); |
| 178 | |
| 179 | NewFunc->setCallingConv(F->getCallingConv()); |
| 180 | F->setCallingConv(CallingConv::SPIR_FUNC); |
| 181 | |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 182 | // Transfer attributes that don't apply to the POD arguments |
| 183 | // to the new functions. |
| 184 | auto Attributes = F->getAttributes(); |
| 185 | SmallVector<std::pair<unsigned, AttributeSet>, 8> AttrBuildInfo; |
| 186 | |
| 187 | // Return attributes have to come first |
| 188 | if (Attributes.hasAttributes(AttributeList::ReturnIndex)) { |
| 189 | auto idx = AttributeList::ReturnIndex; |
| 190 | auto attrs = Attributes.getRetAttributes(); |
| 191 | AttrBuildInfo.push_back(std::make_pair(idx, attrs)); |
| 192 | } |
| 193 | |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame^] | 194 | // Then attributes for non-POD parameters |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 195 | for (auto &rinfo : RemapInfo) { |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame^] | 196 | bool argIsPod = rinfo.arg_kind == clspv::ArgKind::Pod || |
| 197 | rinfo.arg_kind == clspv::ArgKind::PodUBO; |
| 198 | if (!argIsPod && Attributes.hasParamAttrs(rinfo.old_index)) { |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 199 | auto idx = rinfo.new_index + AttributeList::FirstArgIndex; |
| 200 | auto attrs = Attributes.getParamAttributes(rinfo.old_index); |
| 201 | AttrBuildInfo.push_back(std::make_pair(idx, attrs)); |
| 202 | } |
| 203 | } |
| 204 | |
alan-baker | bccf62c | 2019-03-29 10:32:41 -0400 | [diff] [blame] | 205 | // And finally function attributes. |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 206 | if (Attributes.hasAttributes(AttributeList::FunctionIndex)) { |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 207 | auto idx = AttributeList::FunctionIndex; |
alan-baker | bccf62c | 2019-03-29 10:32:41 -0400 | [diff] [blame] | 208 | auto attrs = Attributes.getFnAttributes(); |
| 209 | AttrBuildInfo.push_back(std::make_pair(idx, attrs)); |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 210 | } |
alan-baker | bccf62c | 2019-03-29 10:32:41 -0400 | [diff] [blame] | 211 | auto newAttributes = AttributeList::get(M.getContext(), AttrBuildInfo); |
Kévin Petit | 921c1ab | 2019-03-19 21:25:44 +0000 | [diff] [blame] | 212 | NewFunc->setAttributes(newAttributes); |
| 213 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 214 | // Move OpenCL kernel named attributes. |
| 215 | // TODO(dneto): Attributes starting with kernel_arg_* should be rewritten |
| 216 | // to reflect change in the argument shape. |
| 217 | std::vector<const char *> Metadatas{ |
| 218 | "reqd_work_group_size", "kernel_arg_addr_space", |
| 219 | "kernel_arg_access_qual", "kernel_arg_type", |
| 220 | "kernel_arg_base_type", "kernel_arg_type_qual"}; |
| 221 | for (auto name : Metadatas) { |
| 222 | NewFunc->setMetadata(name, F->getMetadata(name)); |
| 223 | F->setMetadata(name, nullptr); |
| 224 | } |
| 225 | |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 226 | IRBuilder<> Builder(BasicBlock::Create(Context, "entry", NewFunc)); |
| 227 | |
| 228 | // Set kernel argument mapping metadata. |
| 229 | { |
| 230 | // Attach a metadata node named "kernel_arg_map" to the new kernel |
| 231 | // function. It is a tuple of nodes, each of which is a tuple for |
| 232 | // each argument, with members: |
| 233 | // - Argument name |
| 234 | // - Ordinal index in the original kernel function |
| 235 | // - Ordinal index in the new kernel function |
| 236 | // - Byte offset within the argument. This is always 0 for pointer |
| 237 | // arguments. For POD arguments this is the offest within the POD |
| 238 | // argument struct. |
David Neto | 48f56a4 | 2017-10-06 16:44:25 -0400 | [diff] [blame] | 239 | // - Argument type |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 240 | LLVMContext& Context = M.getContext(); |
| 241 | SmallVector<Metadata*, 8> mappings; |
| 242 | for (auto &arg_mapping : RemapInfo) { |
| 243 | auto *name_md = MDString::get(Context, arg_mapping.name); |
| 244 | auto *old_index_md = |
| 245 | ConstantAsMetadata::get(Builder.getInt32(arg_mapping.old_index)); |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 246 | auto *new_index_md = ConstantAsMetadata::get( |
| 247 | Builder.getInt32(arg_mapping.new_index)); |
| 248 | auto *offset_md = |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 249 | ConstantAsMetadata::get(Builder.getInt32(arg_mapping.offset)); |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 250 | auto *arg_size_md = |
| 251 | ConstantAsMetadata::get(Builder.getInt32(arg_mapping.arg_size)); |
Kévin Petit | 8bea15e | 2019-04-09 14:05:17 +0100 | [diff] [blame^] | 252 | auto argKindName = GetArgKindName(arg_mapping.arg_kind); |
| 253 | auto *argtype_md = MDString::get(Context, argKindName); |
David Neto | c6f3ab2 | 2018-04-06 18:02:31 -0400 | [diff] [blame] | 254 | auto *spec_id_md = ConstantAsMetadata::get( |
| 255 | Builder.getInt32(arg_mapping.spec_id)); |
| 256 | auto *arg_md = |
| 257 | MDNode::get(Context, {name_md, old_index_md, new_index_md, |
Kévin PETIT | a353c83 | 2018-03-20 23:21:21 +0000 | [diff] [blame] | 258 | offset_md, arg_size_md, argtype_md, spec_id_md}); |
David Neto | 156783e | 2017-07-05 15:39:41 -0400 | [diff] [blame] | 259 | mappings.push_back(arg_md); |
| 260 | } |
| 261 | |
| 262 | NewFunc->setMetadata("kernel_arg_map", MDNode::get(Context, mappings)); |
| 263 | } |
| 264 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 265 | // Insert the function after the original, to preserve ordering |
| 266 | // in the module as much as possible. |
| 267 | auto &FunctionList = M.getFunctionList(); |
| 268 | for (auto Iter = FunctionList.begin(), IterEnd = FunctionList.end(); |
| 269 | Iter != IterEnd; ++Iter) { |
| 270 | if (&*Iter == F) { |
| 271 | FunctionList.insertAfter(Iter, NewFunc); |
| 272 | break; |
| 273 | } |
| 274 | } |
| 275 | |
| 276 | // The body of the wrapper is essentially a call to the original function, |
| 277 | // but we have to unwrap the non-pointer arguments from the struct. |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 278 | |
| 279 | // Map the wrapper's arguments to the callee's arguments. |
| 280 | SmallVector<Argument *, 8> CallerArgs; |
| 281 | for (Argument &Arg : NewFunc->args()) { |
| 282 | CallerArgs.push_back(&Arg); |
| 283 | } |
| 284 | Argument *PodArg = CallerArgs.back(); |
| 285 | PodArg->setName("podargs"); |
| 286 | |
| 287 | SmallVector<Value *, 8> CalleeArgs; |
| 288 | unsigned podIndex = 0; |
| 289 | unsigned ptrIndex = 0; |
| 290 | for (const Argument &Arg : F->args()) { |
| 291 | if (isa<PointerType>(Arg.getType())) { |
| 292 | CalleeArgs.push_back(CallerArgs[ptrIndex++]); |
| 293 | } else { |
| 294 | CalleeArgs.push_back(Builder.CreateExtractValue(PodArg, {podIndex++})); |
| 295 | } |
| 296 | CalleeArgs.back()->setName(Arg.getName()); |
| 297 | } |
| 298 | assert(ptrIndex + podIndex == F->arg_size()); |
Kévin Petit | 98d9c33 | 2019-03-13 15:03:40 +0000 | [diff] [blame] | 299 | assert(ptrIndex == PtrArgTys.size()); |
| 300 | assert(podIndex != 0); |
| 301 | assert(podIndex == PodArgTys.size()); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 302 | |
| 303 | auto Call = Builder.CreateCall(F, CalleeArgs); |
| 304 | Call->setCallingConv(F->getCallingConv()); |
David Neto | d5b3f98 | 2017-09-28 14:49:49 -0400 | [diff] [blame] | 305 | CallList.push_back(Call); |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 306 | |
| 307 | Builder.CreateRetVoid(); |
| 308 | } |
| 309 | |
David Neto | 482550a | 2018-03-24 05:21:07 -0700 | [diff] [blame] | 310 | // Inline the inner function. It's cleaner to do this. |
| 311 | for (CallInst *C : CallList) { |
| 312 | InlineFunctionInfo info; |
| 313 | Changed |= InlineFunction(C, info); |
David Neto | d5b3f98 | 2017-09-28 14:49:49 -0400 | [diff] [blame] | 314 | } |
| 315 | |
David Neto | dd99221 | 2017-06-23 17:47:55 -0400 | [diff] [blame] | 316 | return Changed; |
| 317 | } |