aaron/ispc
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

AVX bugfix with alignment for store instructions.

Browse Source 
When replacing 'all on' masked store with regular store, set alignment 
to be the vector element alignment, not the alignment for a whole vector. 
(i.e. 4 or 8 byte alignment, not 32 or 64).
This commit is contained in:
Matt Pharr
2011-08-29 16:58:48 -07:00
parent 6de494cfdb
commit ad9e66650d
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
opt.cpp
View File

@@ -637,10 +637,14 @@ IntrinsicsOpt::runOnBasicBlock(llvm::BasicBlock &bb) {
llvm::PointerType::get(storeType, 0), llvm::PointerType::get(storeType, 0),
"ptr2vec", callInst); "ptr2vec", callInst);
lCopyMetadata(castPtr, callInst); lCopyMetadata(castPtr, callInst);
llvm::Instruction *storeInst =
llvm::StoreInst *storeInst =
new llvm::StoreInst(rvalue, castPtr, (llvm::Instruction *)NULL); new llvm::StoreInst(rvalue, castPtr, (llvm::Instruction *)NULL);
int align = callInst->getCalledFunction() == avxMaskedStore32 ? 4 : 8;
storeInst->setAlignment(align);
lCopyMetadata(storeInst, callInst); lCopyMetadata(storeInst, callInst);
llvm::ReplaceInstWithInst(callInst, storeInst); llvm::ReplaceInstWithInst(callInst, storeInst);
modifiedAny = true; modifiedAny = true;
goto restart; goto restart;
} }