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

Dominik Adamski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 13 03:10:02 PDT 2022


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/cfe-commits/attachments/20220913/d5d08ea0/attachment.bin>


More information about the cfe-commits mailing list