FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Allow moddims operations to be part of the JIT tree if possible by umar456 · Pull Request #3177 · arrayfire/arrayfire · GitHub

Repository navigation

Allow moddims operations to be part of the JIT tree if possible - #3177

Merged
9prady9 merged 4 commits into
arrayfire:masterfrom
umar456:moddims_jit
Oct 13, 2021
Merged

9prady9 merged 4 commits into
arrayfire:masterfrom
umar456:moddims_jit

Conversation

umar456 commented Oct 1, 2021 •
edited by 9prady9
Loading

Copy link
Copy Markdown
Member

This PR modifies the moddims function so that it can be included into the JIT tree if the array object is linear. This change makes it possible to perform automatic broadcasting in a future change.

Description

  • Create a moddims node which can be added to a JIT tree if the Array object is linear.
  • Move the moddims function to the common namespace

Changes to Users

Users shouldn't need to make changes to code to take advantage of this change.

Checklist

  • Rebased on latest master
  • Code compiles
  • Tests pass
  • [ ] Functions added to unified API
  • Functions documented

umar456 requested review from 9prady9 and syurkevi October 1, 2021 21:47
Comment thread src/api/c/handle.hpp Outdated
Comment thread src/api/c/optypes.hpp
Comment thread src/backend/common/moddims.hpp Outdated
Comment thread src/backend/cpu/kernel/Array.hpp Outdated
9prady9 mentioned this pull request Oct 11, 2021
2 tasks
9prady9 added this to the 3.8.1 milestone Oct 13, 2021
9prady9 merged commit a800d9f into arrayfire:master Oct 13, 2021
9prady9 pushed a commit to 9prady9/arrayfire that referenced this pull request Oct 13, 2021
…yfire#3177)

* Implement JIT moddims for CUDA and OpenCL

* Create a moddims node instead of modifying setDataDims

* Cleanup headers after moddims change

* Address feedback

(cherry picked from commit a800d9f)
syurkevi pushed a commit that referenced this pull request Dec 28, 2021
* Implement JIT moddims for CUDA and OpenCL

* Create a moddims node instead of modifying setDataDims

* Cleanup headers after moddims change

* Address feedback

(cherry picked from commit a800d9f)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL