This is an archive of the discontinued LLVM Phabricator instance.

Remove unused local variable
AbandonedPublic

Authored by AndyAyers on Nov 9 2015, 7:13 PM.

Details

Reviewers
JosephTremoulet
Summary

Fix warning caused by unused local variable. NFC.

Diff Detail

Repository
rL LLVM

Event Timeline

AndyAyers updated this revision to Diff 39779.Nov 9 2015, 7:13 PM
AndyAyers retitled this revision from to Remove unused local variable.
AndyAyers updated this object.
AndyAyers set the repository for this revision to rL LLVM.
AndyAyers added a reviewer: JosephTremoulet.
AndyAyers added a subscriber: llvm-commits.

committed something like this along with another fix in 252580

Feel free to just commit stuff like this without review in the future, if
you have commit access - it's simple enough.

Or just not cause the problem in the first place... not sure why the builds I did didn’t flag this.

From: David Blaikie [mailto:dblaikie@gmail.com]
Sent: Monday, November 9, 2015 7:20 PM
To: reviews+D14525+public+44dfeb4c191f02ac@reviews.llvm.org; Andy Ayers <andya@microsoft.com>
Cc: llvm-commits <llvm-commits@lists.llvm.org>
Subject: Re: [PATCH] D14525: Remove unused local variable

committed something like this along with another fix in 252580

Feel free to just commit stuff like this without review in the future, if you have commit access - it's simple enough.

AndyAyers abandoned this revision.Nov 9 2015, 7:38 PM

Fixed by 252580