This is an archive of the discontinued LLVM Phabricator instance.

Quiet unused variable warnings in Release builds
ClosedPublic

Authored by DavidKreitzer on Mar 16 2018, 1:55 PM.

Diff Detail

Event Timeline

DavidKreitzer created this revision.Mar 16 2018, 1:55 PM
craig.topper accepted this revision.Mar 16 2018, 2:11 PM

LGTM

What compiler is warning on this? I thought the build bots usually catch this, but maybe the assignments inside the loop is making some compilers not notice?

This revision is now accepted and ready to land.Mar 16 2018, 2:11 PM

Thanks, Craig. I noticed it with gcc 4.8.5.

This revision was automatically updated to reflect the committed changes.

Thanks David for spotting and fixing this!