search

clang-omp / clang

clang with OpenMP 3.1 and some elements of OpenMP 4.0 support
clang-omp.github.com
Other
91 stars 15 forks source link

copy-and-paste error in TreeTransform<Derived>::TransformOMPNumTeamsClause #24

Closed hfinkel closed 10 years ago

hfinkel commented 10 years ago

The code in TreeTransform::TransformOMPNumTeamsClause calls getDerived().RebuildOMPSimdlenClause -- this seems wrong.

template OMPClause TreeTransform::TransformOMPNumTeamsClause(OMPNumTeamsClause C) { // Transform the number-of-teams expession. ExprResult E = getDerived().TransformExpr(C->getNumTeams());

if (E.isInvalid()) return 0;

return getDerived().RebuildOMPSimdlenClause(E.take(), C->getLocStart(), C->getLocEnd()); }

alexey-bataev commented 10 years ago

Hal, thanks, I'll fix that

alexey-bataev commented 10 years ago

Fixed.