This is an archive of the discontinued LLVM Phabricator instance.

[clang-doc] Removing -Wunused-variable warning
ClosedPublic

Authored by juliehockett on Mar 26 2018, 3:15 PM.

Reviewers

Commits

Summary

Warning was appearing in release with debug info build, this removes it.

LGTM

This revision is now accepted and ready to land.Mar 26 2018, 3:18 PM

LGTM.

This revision was automatically updated to reflect the committed changes.

I think it's fine to commit trivial fixes without going via review.

Path

Size

clang-tools-extra/

trunk/

clang-doc/

4 lines

	Show First 20 Lines • Show All 258 Lines • ▼ Show 20 Lines
	void ClangDocBitcodeWriter::emitBlockID(BlockId BID) {			void ClangDocBitcodeWriter::emitBlockID(BlockId BID) {
	const auto &BlockIdName = BlockIdNameMap[BID];			const auto &BlockIdName = BlockIdNameMap[BID];
	assert(BlockIdName.data() && BlockIdName.size() && "Unknown BlockId.");			assert(BlockIdName.data() && BlockIdName.size() && "Unknown BlockId.");

	Record.clear();			Record.clear();
	Record.push_back(BID);			Record.push_back(BID);
	Stream.EmitRecord(llvm::bitc::BLOCKINFO_CODE_SETBID, Record);			Stream.EmitRecord(llvm::bitc::BLOCKINFO_CODE_SETBID, Record);
	Stream.EmitRecord(llvm::bitc::BLOCKINFO_CODE_BLOCKNAME,			Stream.EmitRecord(llvm::bitc::BLOCKINFO_CODE_BLOCKNAME,
	ArrayRef<unsigned char>(BlockIdNameMap[BID].bytes_begin(),			ArrayRef<unsigned char>(BlockIdName.bytes_begin(),
	BlockIdNameMap[BID].bytes_end()));			BlockIdName.bytes_end()));
	}			}

	/// \brief Emits a record name to the BLOCKINFO block.			/// \brief Emits a record name to the BLOCKINFO block.
	void ClangDocBitcodeWriter::emitRecordID(RecordId ID) {			void ClangDocBitcodeWriter::emitRecordID(RecordId ID) {
	assert(RecordIdNameMap[ID] && "Unknown RecordId.");			assert(RecordIdNameMap[ID] && "Unknown RecordId.");
	prepRecordData(ID);			prepRecordData(ID);
	Record.append(RecordIdNameMap[ID].Name.begin(),			Record.append(RecordIdNameMap[ID].Name.begin(),
	RecordIdNameMap[ID].Name.end());			RecordIdNameMap[ID].Name.end());
	▲ Show 20 Lines • Show All 234 Lines • Show Last 20 Lines