This is an archive of the discontinued LLVM Phabricator instance.

[CodeGen] Fix a warning in DAGTypeLegalizer::SetSplitVector
ClosedPublic

Authored by david-arm on Jul 6 2020, 1:51 AM.

Reviewers

Commits

Summary

This fixes up one warning in the test:

sve-sext-zext.ll

	Time	Test
	50 ms	linux > Polly.Isl/CodeGen/MemAccess::Unknown Unit Message ("")
	90 ms	linux > Polly.ScopInfo::Unknown Unit Message ("")
	10,460 ms	linux > libomp.env::Unknown Unit Message ("")
	1,470 ms	linux > libomp.worksharing/for::Unknown Unit Message ("")

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

LGTM

This revision is now accepted and ready to land.Jul 9 2020, 2:53 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

lib/

CodeGen/

SelectionDAG/

4 lines

Context not available.
	SDValue Hi) {	SDValue Hi) {
	assert(Lo.getValueType().getVectorElementType() ==	assert(Lo.getValueType().getVectorElementType() ==
	Op.getValueType().getVectorElementType() &&	Op.getValueType().getVectorElementType() &&
		Lint: Pre-merge checks Inline Actions clang-format: please reformat the code - Op.getValueType().getVectorElementType() && + Op.getValueType().getVectorElementType() && Lint: Pre-merge checks: clang-format: please reformat the code ``` - Op.getValueType().getVectorElementType()…
	2*Lo.getValueType().getVectorNumElements() ==	Lo.getValueType().getVectorElementCount() * 2 ==
	Op.getValueType().getVectorNumElements() &&	Op.getValueType().getVectorElementCount() &&
		Lint: Pre-merge checks Inline Actions clang-format: please reformat the code - Op.getValueType().getVectorElementCount() && + Op.getValueType().getVectorElementCount() && Lint: Pre-merge checks: clang-format: please reformat the code ``` - Op.getValueType().getVectorElementCount()…
	Hi.getValueType() == Lo.getValueType() &&	Hi.getValueType() == Lo.getValueType() &&
	"Invalid type for split vector");	"Invalid type for split vector");
	// Lo/Hi may have been newly allocated, if so, add nodeid's as relevant.	// Lo/Hi may have been newly allocated, if so, add nodeid's as relevant.
Context not available.