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

[Dominik Adamski via Phabricator via llvm-commits]

domada updated this revision to Diff 459697.
domada added a comment.

Applied review remarks. Scope of changes:

1. replaced ArrayRef<Value*> AlignedVars, Value * Alignment arguments with DenseMap. It allows to generate different alignment assumption for different variables. OpenMP allows to specify multiple `aligned` clauses for single SIMD construct.
2. modified calculation of default alignment. Use pointer ABI alignment as the default one.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133578/new/

https://reviews.llvm.org/D133578

Files:
  clang/lib/CodeGen/CGStmtOpenMP.cpp
  llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
  llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
  llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
  mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D133578.459697.patch
Type: text/x-patch
Size: 13558 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220913/d5d08ea0/attachment.bin>