This is an archive of the discontinued LLVM Phabricator instance.

[Polly] Clang complains about missing virtual destructor
ClosedPublic

Authored by gchatelet on Jul 16 2019, 12:50 AM.

Reviewers

Commits

Summary

Feel free to reassign if needed.

Repository

rG LLVM Github Monorepo

Build Status

Buildable 35069
Build 35068: arc lint + arc unit

Herald added a project: Restricted Project. · View Herald Transcript

A gentle ping. It's a one liner and should be reviewed easily :)

LGTM

This revision is now accepted and ready to land.Jul 22 2019, 7:58 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

polly/

include/

polly/

CodeGen/

2 lines

Show First 20 Lines • Show All 112 Lines • ▼ Show 20 Lines	public:
/// Create a parallel loop generator for the current function.		/// Create a parallel loop generator for the current function.
ParallelLoopGenerator(PollyIRBuilder &Builder, LoopInfo &LI,		ParallelLoopGenerator(PollyIRBuilder &Builder, LoopInfo &LI,
DominatorTree &DT, const DataLayout &DL)		DominatorTree &DT, const DataLayout &DL)
: Builder(Builder), LI(LI), DT(DT),		: Builder(Builder), LI(LI), DT(DT),
LongType(		LongType(
Type::getIntNTy(Builder.getContext(), DL.getPointerSizeInBits())),		Type::getIntNTy(Builder.getContext(), DL.getPointerSizeInBits())),
M(Builder.GetInsertBlock()->getParent()->getParent()) {}		M(Builder.GetInsertBlock()->getParent()->getParent()) {}

		virtual ~ParallelLoopGenerator() {}

/// Create a parallel loop.		/// Create a parallel loop.
///		///
/// This function is the main function to automatically generate a parallel		/// This function is the main function to automatically generate a parallel
/// loop with all its components.		/// loop with all its components.
///		///
/// @param LB The lower bound for the loop we parallelize.		/// @param LB The lower bound for the loop we parallelize.
/// @param UB The upper bound for the loop we parallelize.		/// @param UB The upper bound for the loop we parallelize.
/// @param Stride The stride of the loop we parallelize.		/// @param Stride The stride of the loop we parallelize.
▲ Show 20 Lines • Show All 88 Lines • Show Last 20 Lines