Juan Manuel Martinez =?utf-8?q?Caamaño?= <juama...@amd.com>, Juan Manuel Martinez =?utf-8?q?Caamaño?= <juama...@amd.com>, Juan Manuel Martinez =?utf-8?q?Caamaño?= <juama...@amd.com> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/133...@github.com>
================ @@ -260,7 +260,7 @@ AMDGPUTargetInfo::AMDGPUTargetInfo(const llvm::Triple &Triple, MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 64; CUMode = !(GPUFeatures & llvm::AMDGPU::FEATURE_WGP); - for (auto F : {"image-insts", "gws"}) + for (auto F : {"image-insts", "gws", "mem-to-lds-load-insts"}) ---------------- arsenm wrote: ```suggestion for (auto F : {"image-insts", "gws", "vmem-to-lds-load-insts"}) ``` Not sure about the naming. The manual refers to this as "LDS DMA", but was new in gfx8 (or maybe, the store case was new? I thought the load case always existed, we just never made use of it). https://github.com/llvm/llvm-project/pull/133055 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits