This is an archive of the discontinued LLVM Phabricator instance.

[Bugpoint] fix use-after-move. NFC
ClosedPublic

Authored by nickdesaulniers on May 19 2019, 6:34 PM.

Details

Summary

This was flagged in https://www.viva64.com/en/b/0629/ under "Snippet No.
6".

Note that author also states:
"Note that the loop doesn't actually execute at all."

This is not true, but the author can be forgiven; there's two distinct
variables with very similar identifiers:

MiscompiledFunctions
MisCompFunctions

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMay 19 2019, 6:34 PM
RKSimon accepted this revision.May 21 2019, 2:52 AM

LGTM - cheers

This revision is now accepted and ready to land.May 21 2019, 2:52 AM
This revision was automatically updated to reflect the committed changes.