Skip to content

Conversation

@ZuseZ4
Copy link
Collaborator

@ZuseZ4 ZuseZ4 commented Jul 2, 2023

No description provided.

@ZuseZ4 ZuseZ4 changed the title add spmv rules for x,beta,y add spmv rule Jul 2, 2023
@ZuseZ4 ZuseZ4 marked this pull request as ready for review July 30, 2023 03:32
@ZuseZ4
Copy link
Collaborator Author

ZuseZ4 commented Aug 2, 2023

I think I got the indexing right, but here I'd really prefer some actual julia tests that run against FD or others.

@ZuseZ4 ZuseZ4 requested a review from wsmoses August 2, 2023 22:33
@ZuseZ4 ZuseZ4 enabled auto-merge (squash) August 2, 2023 22:58
llvm::Type *BlasFPT, llvm::Type *BlasPT,
llvm::Type *BlasIT, llvm::Type *fpTy,
llvm::ArrayRef<llvm::Value *> args,
const llvm::ArrayRef<llvm::OperandBundleDef> bundles,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
const llvm::ArrayRef<llvm::OperandBundleDef> bundles,
llvm::ArrayRef<llvm::OperandBundleDef> bundles,

result.append((Twine("d_") + name).str());
} else {
result.append((Twine("arg_") + name).str());
// result.append((Twine("fp_") + name).str());
Copy link
Member

Choose a reason for hiding this comment

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

do we still need this?

Co-authored-by: Tim Gymnich <tgymnich@icloud.com>
@wsmoses wsmoses disabled auto-merge August 5, 2023 07:22
@wsmoses wsmoses merged commit 41693dc into main Aug 5, 2023
@wsmoses wsmoses deleted the tg-blas-spmv branch August 5, 2023 07:22
@tgymnich
Copy link
Member

tgymnich commented Aug 8, 2023

it looks like this PR broke macOS CI

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