Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: vfmaddsub and friends for complex matmul #150

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Chronum94
Copy link

This is still incredibly WIP, and it's in the context of #140. I'm not really sure on how to proceed here, so dumping this here in case @chriselrod can see a way forward.

@chriselrod
Copy link
Collaborator

I'm not really sure on how to proceed here

Have any questions/anything blocking you?

@Chronum94
Copy link
Author

Roughly in order of priority:

  1. What is the correct way of selecting the correct calls depending on the value of theta and eta? I tried macros and inline functions, but I was likely doing something wrong which is why I got error messages that were a bit too dense for me (I can paste them here if it can potentially help).
  2. How bad (tentatively) are the extra sign multiplications going to affect things?
  3. The final conjugation of _C/C, I'm really not sure how to incorporate that with the new unreshaped C array, so I don't really know what to ask here other than 'how do?'

signs = Vec(ntuple(x -> ifelse(iseven(x), -one(T), one(T)), pick_vector_width(Float64))...)
if !η_bool & !θ_bool
cmatmul_ab(C, A, B)

Copy link
Collaborator

@chriselrod chriselrod Aug 1, 2022

Choose a reason for hiding this comment

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

Suggested change
end

maybe?
I just glanced at the code to see if there was any obvious cause to the error.

Copy link
Author

Choose a reason for hiding this comment

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

Yeah this is all still very WIP. I'll work on this again tomorrow (hard maybe), just wanted to get this pushed since was switching around on machines.

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.

2 participants