This is an archive of the discontinued LLVM Phabricator instance.

ScopInfo: Replace while/iterator construct with std::remove_if
ClosedPublic

Authored by grosser on Nov 29 2015, 9:36 PM.

Details

Summary

The use of C++'s high-level iterator functionality instead of two while loops
and explicit iterator handling improves readability of this code.

This passes LNT -polly-process-unprofitable for me.

Proposed-by: Michael Kruse <llvm@meinersbur.de>

Diff Detail

Repository
rL LLVM

Event Timeline

grosser updated this revision to Diff 41367.Nov 29 2015, 9:36 PM
grosser retitled this revision from to ScopInfo: Replace while/iterator construct with std::remove_if.
grosser updated this object.
grosser added reviewers: jdoerfert, Meinersbur.
grosser added subscribers: llvm-commits, pollydev.
This revision was automatically updated to reflect the committed changes.