[PATCH] D133578: [OpenMP] Add generation of SIMD align assumptions to OMPIRBuilder

domada marked an inline comment as done. domada added inline comments. Comment at: llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h:629 + llvm::ArrayRef AlignedVars, + llvm::Value *Alignment, Value *IfCond, ConstantInt *Simdlen, C

[PATCH] D133578: [OpenMP] Add generation of SIMD align assumptions to OMPIRBuilder

domada updated this revision to Diff 459697. domada added a comment. Applied review remarks. Scope of changes: 1. replaced ArrayRef AlignedVars, Value * Alignment arguments with DenseMap. It allows to generate different alignment assumption for different variables. OpenMP allows to specify mult

[PATCH] D133578: [OpenMP] Add generation of SIMD align assumptions to OMPIRBuilder

jdoerfert added a comment. Thanks for this patch! I have two drive by comments that should probably be addressed first. Comment at: llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h:629 + llvm::ArrayRef AlignedVars, + llvm::Value *Alignment, Valu

[PATCH] D133578: [OpenMP] Add generation of SIMD align assumptions to OMPIRBuilder

domada created this revision. domada added a reviewer: kiranchandramohan. Herald added subscribers: bzcheeseman, awarzynski, sdasgup3, wenzhicui, wrengr, cota, teijeong, rdzhabarov, tatianashp, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, mgester, arpith-jacob, antiagainst,