Skip to content

[HINT] [Shared_Memory] Support hint the use of shared memory for aipu#29

Merged
zhzhcookie merged 12 commits into
mainfrom
shared_memory_20251015
Nov 7, 2025
Merged

[HINT] [Shared_Memory] Support hint the use of shared memory for aipu#29
zhzhcookie merged 12 commits into
mainfrom
shared_memory_20251015

Conversation

@youngyoung01

Copy link
Copy Markdown
Collaborator

No description provided.

youngyoung01 and others added 12 commits October 15, 2025 09:45
* When using shared memory, the annotation "@hint: shared_memory" is required.
* MemorySpace value 8 of shared memory is only for AIPU backend.
 test_02_fused_softmax.py requires support later.
 At present in the use of Shared memory, need to comment out mlir/lib/the Dialect/Affine/Analysis/LoopAnalysis.cpp
 in the following two lines, and recompile LLVM:

 //if (! memRefType.getLayout().isIdentity())
 //  return memoryOp.emitError("NYI: non-trivial layout map"), false;
 Fixed the bug that caused errors in the operation of 2D matrix addition.
…d" attribute.

 * The extension supports multi-dimensional vector add.
 * Add the "shared" attribute to the memref.subview carrying offset of shared memory
   to solve the problem of non-trivial layout.
…d memory address space

 * Conflicts with DMA hints need to be resolved during merging.

@zhzhcookie zhzhcookie left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zhzhcookie zhzhcookie changed the title [Shared_Memory] Support the use of shared memory for AIPU. [HINT] [Shared_Memory] Support hint the use of shared memory for aipu Nov 7, 2025
@zhzhcookie zhzhcookie merged commit 9d1b5e0 into main Nov 7, 2025
1 check passed
@zhzhcookie zhzhcookie deleted the shared_memory_20251015 branch November 7, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants