[qt/clang/llvm]: Summary of bulk changes made
KDE Git Services - Bulk Change <[email protected]>
| Newsgroups | gmane.comp.kde.cvs |
|---|---|
| Message-ID | <[email protected]> |
Git repository change summary for qt/clang/llvm Pushed by mirror-service into branch 'upstream/users/abhinavgaba/attach-maps-in-mappers-3'. Changed from 8533f3d94f973576dc816be9f54708ff993f2cf6 to cd434d71e9646c7979729979bfe16fa6dd7875a8 Acknowledgement was received that this change introduces only existing code that has been pushed to another public open source repository. This change contains the following new commits: Git commit acafa3ccdc95732cb5781c0e733dc1899c18c79f by Abhinav Gaba on 27/07/2026 at 18:46.. [OpenMP] Propagate ALWAYS/DELETE/CLOSE map-type modifiers to mapper entries When a map clause uses a user-defined mapper, the map-type-modifying bits (ALWAYS, DELETE, CLOSE) on the outer clause must apply to each map the mapper inserts (OpenMP 6.0:281:34). Propagate them in emitUserDefinedMapper by OR-ing the imported modifier bits into each pushed component, except ATTACH entries (ATTACH|ALWAYS is reserved for attach(always) and the other bits are meaningless for ATTACH). PRESENT is intentionally not propagated here yet (a TODO notes it is handled in a follow-on, since it requires distinguishing pointee entries from the struct's own storage). Update offload mapper_map_always_from.c to its now-correct behavior: ALWAYS forces a member copy-back that the ref count would otherwise suppress, so s.y reads back as 111 (it was 0 before this change). Co-Authored-By: Claude Opus 4.8 <[email protected]> https://invent.kde.org/qt/clang/llvm/-/commit/acafa3ccdc95732cb5781c0e733dc1899c18c79f Git commit cd434d71e9646c7979729979bfe16fa6dd7875a8 by Abhinav Gaba on 27/07/2026 at 18:47.. [OpenMP][Clang] Enable ATTACH-style maps for mappers. Track per-entry attach-ptr info (HasAttachPtr) through mapper codegen so that emitUserDefinedMapper does not add a new outer MEMBER_OF to pointee/combined entries (which occupy different storage than the struct) or to ATTACH entries. Clang and the MLIR translator populate the per-entry array in parallel with the other MapInfosTy arrays. Address review: - Rename MapSkipMemberOfArrayTy to MapHasAttachPtrArrayTy to match the HasAttachPtr field it backs. - Restructure the emitUserDefinedMapper comment into a bulleted (*)/(**)/(***) list keyed to the example entries. - Reword the Clang comments: HasAttachPtr marks pointee entries that have a base attach-ptr; a combined entry has a base attach-ptr if its constituents do; cross-reference emitUserDefinedMapper for the MEMBER_OF rationale. - Update the moved present-check tests to their now-correct behavior (the attach-style maps make the inbounds present checks pass and remove the "explicit extension" errors). Co-Authored-By: Claude Opus 4.8 <[email protected]> https://invent.kde.org/qt/clang/llvm/-/commit/cd434d71e9646c7979729979bfe16fa6dd7875a8