kiranchandramohan added a comment. In D129149#3632861 <https://reviews.llvm.org/D129149#3632861>, @psoni2628 wrote:
> In D129149#3632813 <https://reviews.llvm.org/D129149#3632813>, > @kiranchandramohan wrote: > >> Nit: Also add to the summary that this patch uses the simdlen support in >> OpenMPIRBuilder when it is enabled in Clang. > > I just wanted to clarify that below is what you mean? > > This patch adds OMPIRBuilder support for the simdlen clause for the > simd directive. It uses the simdlen support in OpenMPIRBuilder when > it is enabled in clang. Yes. Thanks. Maybe you can also add that simdlen is lowered by the OpenMPIRBuilder by generating the loop.vectorize.width metadata. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129149/new/ https://reviews.llvm.org/D129149 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
