================
@@ -735,11 +736,15 @@ wrapDeviceImages(ArrayRef<std::unique_ptr<MemoryBuffer>> 
Buffers,
 }
 
 Expected<SmallVector<std::unique_ptr<MemoryBuffer>>>
-bundleOpenMP(ArrayRef<OffloadingImage> Images) {
+bundleOpenMP(SmallVectorImpl<OffloadingImage> &Images) {
----------------
sarnex wrote:

It's mutable because we modify it inside `bundleOpenMP` if the vendor is 
`Intel`, where we call `containerizeOpenMPSPIRVImage` to put the device image 
into an ELF (which we can't do using clang because there's no SPIR-V linker). I 
agree it's not ideal to have to modify the image here, if you have a better 
spot to do that ELF packaging, let me know. Thanks

https://github.com/llvm/llvm-project/pull/120145
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to