Re: [PATCH] [OPENMP] Codegen of the ‘aligned’ clause for the ‘omp simd’ directive

hfinkel at anl.gov hfinkel at anl.gov
Mon Sep 29 10:30:24 PDT 2014


> I added type parameter to getOpenMPSimdDefaultAlignment (I guess you meant llvm::Type, not clang::Type) 

I think I'd actually like the QualType (which I suppose is just E->getType()), so that pointers to single-element structures can be handled like pointers to the underlying type in a straightforward way (meaning using the same logic that already exists in the ABI code).

Otherwise, LGTM. Thanks!

http://reviews.llvm.org/D5499






More information about the cfe-commits mailing list