This is an archive of the discontinued LLVM Phabricator instance.

[flang] Fix -Wunused-variable in -DLLVM_ENABLE_ASSERTIONS=off build
ClosedPublic

Authored by peixin on Apr 9 2022, 7:46 AM.

Reviewers

Commits

Summary

Fix the warning from D122483.

peixin requested review of this revision.Apr 9 2022, 7:46 AM

LGTM.

This revision is now accepted and ready to land.Apr 9 2022, 7:47 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

flang/

lib/

Lower/

4 lines

Show First 20 Lines • Show All 216 Lines • ▼ Show 20 Lines	if (const auto &ifClause =
}		}
procBindKindAttr =		procBindKindAttr =
omp::ClauseProcBindKindAttr::get(firOpBuilder.getContext(), pbKind);		omp::ClauseProcBindKindAttr::get(firOpBuilder.getContext(), pbKind);
} else if (const auto &allocateClause =		} else if (const auto &allocateClause =
std::get_if<Fortran::parser::OmpClause::Allocate>(		std::get_if<Fortran::parser::OmpClause::Allocate>(
&clause.u)) {		&clause.u)) {
genAllocateClause(converter, allocateClause->v, allocatorOperands,		genAllocateClause(converter, allocateClause->v, allocatorOperands,
allocateOperands);		allocateOperands);
} else if (const auto &privateClause =		} else if (std::get_if<Fortran::parser::OmpClause::Private>(&clause.u)) {
std::get_if<Fortran::parser::OmpClause::Private>(
&clause.u)) {
// TODO: Handle private. This cannot be a hard TODO because testing for		// TODO: Handle private. This cannot be a hard TODO because testing for
// allocate clause requires private variables.		// allocate clause requires private variables.
} else {		} else {
TODO(currentLocation, "OpenMP Block construct clauses");		TODO(currentLocation, "OpenMP Block construct clauses");
}		}
}		}

for (const auto &clause :		for (const auto &clause :
▲ Show 20 Lines • Show All 186 Lines • Show Last 20 Lines