This is an archive of the discontinued LLVM Phabricator instance.

Differential D64389

[NFC] [X86] Fix scan-build complaining
ClosedPublic

Authored by pengfei on Jul 9 2019, 12:27 AM.

Details

Reviewers
RKSimon
xiangzhangllvm
craig.topper
Commits
rGa50bbfc47027: [NFC] [X86] Fix scan-build complaining
rL365473: [NFC] [X86] Fix scan-build complaining
rC365473: [NFC] [X86] Fix scan-build complaining
Summary

Remove unused variable. This fixes bug:
https://bugs.llvm.org/show_bug.cgi?id=42526

Signed-off-by: pengfei <pengfei.wang@intel.com>

Diff Detail

Event Timeline

pengfei created this revision.Jul 9 2019, 12:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2019, 12:27 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Harbormaster completed remote builds in B34567: Diff 208605.Jul 9 2019, 12:29 AM
RKSimon accepted this revision.Jul 9 2019, 5:25 AM

LGTM - cheers

This revision is now accepted and ready to land.Jul 9 2019, 5:25 AM
Closed by commit rL365473: [NFC] [X86] Fix scan-build complaining (authored by pengfei). · Explain WhyJul 9 2019, 5:41 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2019, 5:41 AM
Herald added a subscriber: llvm-commits. · View Herald Transcript

Revision Contents

PathSize
clang/
lib/
CodeGen/
5 lines

Diff 208605

clang/lib/CodeGen/CGBuiltin.cpp

  • This file is larger than 256 KB, so syntax highlighting is disabled by default.
Show First 20 Lines • Show All 11,749 Lines • ▼ Show 20 Lines case X86::BI__builtin_ia32_vp2intersect_d_128: {
case X86::BI__builtin_ia32_vp2intersect_d_128: case X86::BI__builtin_ia32_vp2intersect_d_128:
ID = Intrinsic::x86_avx512_vp2intersect_d_128; ID = Intrinsic::x86_avx512_vp2intersect_d_128;
break; break;
} }
Value *Call = Builder.CreateCall(CGM.getIntrinsic(ID), {Ops[0], Ops[1]}); Value *Call = Builder.CreateCall(CGM.getIntrinsic(ID), {Ops[0], Ops[1]});
Value *Result = Builder.CreateExtractValue(Call, 0); Value *Result = Builder.CreateExtractValue(Call, 0);
Result = EmitX86MaskedCompareResult(*this, Result, NumElts, nullptr); Result = EmitX86MaskedCompareResult(*this, Result, NumElts, nullptr);
Value *Store = Builder.CreateDefaultAlignedStore(Result, Ops[2]); Builder.CreateDefaultAlignedStore(Result, Ops[2]);
Result = Builder.CreateExtractValue(Call, 1); Result = Builder.CreateExtractValue(Call, 1);
Result = EmitX86MaskedCompareResult(*this, Result, NumElts, nullptr); Result = EmitX86MaskedCompareResult(*this, Result, NumElts, nullptr);
Store = Builder.CreateDefaultAlignedStore(Result, Ops[3]); return Builder.CreateDefaultAlignedStore(Result, Ops[3]);
return Store;
} }
case X86::BI__builtin_ia32_vpmultishiftqb128: case X86::BI__builtin_ia32_vpmultishiftqb128:
case X86::BI__builtin_ia32_vpmultishiftqb256: case X86::BI__builtin_ia32_vpmultishiftqb256:
case X86::BI__builtin_ia32_vpmultishiftqb512: { case X86::BI__builtin_ia32_vpmultishiftqb512: {
Intrinsic::ID ID; Intrinsic::ID ID;
switch (BuiltinID) { switch (BuiltinID) {
default: llvm_unreachable("Unsupported intrinsic!"); default: llvm_unreachable("Unsupported intrinsic!");
▲ Show 20 Lines • Show All 2,547 Lines • Show Last 20 Lines