This is an archive of the discontinued LLVM Phabricator instance.

[SVE] Add checks for no warnings in CodeGen/AArch64/sve-sext-zext.ll
ClosedPublic

Authored by david-arm on Jul 6 2020, 2:54 AM.

Download Raw Diff

Details

Reviewers

rengolin
efriedma
sdesmalen
kmclaughlin

Commits

rGf43b5c7a76ab: [SVE] Add checks for no warnings in CodeGen/AArch64/sve-sext-zext.ll

Summary

Previous patches fixed up all the warnings in this test:

llvm/test/CodeGen/AArch64/sve-sext-zext.ll

and this change simply checks that no new warnings are added in future.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

david-arm created this revision.Jul 6 2020, 2:54 AM

Herald added a reviewer: rengolin. · View Herald TranscriptJul 6 2020, 2:54 AM

Herald added a reviewer: efriedma. · View Herald Transcript

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

Herald added subscribers: llvm-commits, danielkiss, psnobl and 2 others. · View Herald Transcript

david-arm added parent revisions: D83198: [CodeGen] Remove calls to getVectorNumElements in DAGTypeLegalizer::SplitVecOp_EXTRACT_SUBVECTOR, D83197: [CodeGen] Fix warning in DAGTypeLegalizer::SplitVecRes_ExtendOp, D83203: [CodeGen] Fix warnings in SelectionDAG::SplitVector, D83195: [CodeGen] Fix a warning in DAGTypeLegalizer::PromoteIntRes_EXTRACT_SUBVECTOR, D83196: [CodeGen] Fix a warning in DAGTypeLegalizer::SetSplitVector.Jul 6 2020, 2:56 AM

david-arm added reviewers: sdesmalen, kmclaughlin.

Harbormaster failed remote builds in B62980: Diff 275628!Jul 6 2020, 3:42 AM

david-arm updated this revision to Diff 275925.Jul 6 2020, 11:56 PM

LGTM

This revision is now accepted and ready to land.Jul 7 2020, 12:04 AM

Closed by commit rGf43b5c7a76ab: [SVE] Add checks for no warnings in CodeGen/AArch64/sve-sext-zext.ll (authored by david-arm). · Explain WhyJul 29 2020, 5:08 AM

This revision was automatically updated to reflect the committed changes.

david-arm added a commit: rGf43b5c7a76ab: [SVE] Add checks for no warnings in CodeGen/AArch64/sve-sext-zext.ll.

Revision Contents

Path

Size

llvm/

test/

CodeGen/

AArch64/

sve-sext-zext.ll

5 lines

Diff 281529

llvm/test/CodeGen/AArch64/sve-sext-zext.ll

	; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py			; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
	; RUN: llc -mtriple=aarch64--linux-gnu -mattr=+sve < %s \| FileCheck %s			; RUN: llc -mtriple=aarch64--linux-gnu -mattr=+sve < %s 2>%t \| FileCheck %s
				; RUN: FileCheck --check-prefix=WARN --allow-empty %s <%t

				; WARN-NOT: warning

	define <vscale x 16 x i8> @sext_i1_i8(<vscale x 16 x i1> %a) {			define <vscale x 16 x i8> @sext_i1_i8(<vscale x 16 x i1> %a) {
	; CHECK-LABEL: sext_i1_i8:			; CHECK-LABEL: sext_i1_i8:
	; CHECK: // %bb.0:			; CHECK: // %bb.0:
	; CHECK-NEXT: mov z0.b, p0/z, #-1 // =0xffffffffffffffff			; CHECK-NEXT: mov z0.b, p0/z, #-1 // =0xffffffffffffffff
	; CHECK-NEXT: ret			; CHECK-NEXT: ret
	%r = sext <vscale x 16 x i1> %a to <vscale x 16 x i8>			%r = sext <vscale x 16 x i1> %a to <vscale x 16 x i8>
	ret <vscale x 16 x i8> %r			ret <vscale x 16 x i8> %r
	▲ Show 20 Lines • Show All 318 Lines • Show Last 20 Lines