Change Details

This patch extends the types on which the align parameter attribute is permissible to include vectors of pointers. It does so in order to allow alignment specification on llvm.vp.gather and llvm.vp.scatter intrinsics, which lack an explicit alignment parameter. The default alignment of these intrinsics was previously documented as being equal to the ABI alignment of the *scalar* type, when in fact that wasn't the case: the ABI alignment of the vector type was used instead. This has also been fixed in this patch.

This patch fixes a case where the 'align' parameter attribute on the pointer operands to llvm.vp.gather and llvm.vp.scatter was being dropped during the conversion to the SelectionDAG. The default alignment equal to the ABI type alignment of the vector type was kept. It also updates the documentation to reflect the fact that the parameter attribute is now properly supported. The default alignment of these intrinsics was previously documented as being equal to the ABI alignment of the *scalar* type, when in fact that wasn't the case: the ABI alignment of the vector type was used instead. This has also been fixed in this patch.

This patch extends the types on whichfixes a case where the 'align' parameter attribute ison the permissiblepointer operands to include vectors of pointers. It does so in order to allowllvm.vp.gather and llvm.vp.scatter was being dropped alignment specificatduring the conversion on llvm.vp.gather and llvm.vp.scatterto the SelectionDAG. The default alignment equal intrinsics,to the ABI type alignment of the vector type was kept. which lack an explicit alignment parameterIt also updates the documentation to reflect the fact that the parameter attribute is now properly supported. The default alignment of these intrinsics was previously documented as being equal to the ABI alignment of the *scalar* type, when in fact that wasn't the case: the ABI alignment of the vector type was used instead. This has also been fixed in this patch.