[PATCH] D138496: [WIP][OMPIRBuilder] Add OpenMPDefaultSimdAlignment field to TargetMachine class

Jan Sjödin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 07:22:23 PST 2022


jsjodin added a comment.

In D138496#3944348 <https://reviews.llvm.org/D138496#3944348>, @domada wrote:

> Yes, we can extend calculations of the default alignment. My primary goal was to move code from clang into llvm without modification of the code logic.
>
> AFAIK the default simd alignment corresponds to the maximal width of the simd instruction for given target.

So maybe the function should be called getMaximumVectorTypeAlignment or something like that, and query the target with some relevant types to find the maximum, alternatively implement this at a lower level inside DataLayout, and not have to provide a type.


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

https://reviews.llvm.org/D138496



More information about the llvm-commits mailing list