This is an archive of the discontinued LLVM Phabricator instance.

Minor cleanup to PartialInlining logic.
Needs ReviewPublic

Authored by tulloch on Apr 3 2014, 6:57 AM.



Two minor changes:

  • Replaces a while loop over basic blocks with that unconditionally increments the counter with the equivalent for loop. This makes the scope of the loop variable slightly clearer (given this is a ~90 line function).
  • Replace an instance of std::vector<User *> with std::vector<User*> to be conformant with style guide and consistent with surrounding code.

Diff Detail

Event Timeline

chandlerc resigned from this revision.Mar 29 2015, 11:44 AM
chandlerc removed a reviewer: chandlerc.

Sorry we never got to this.

If you want to pursue it, please update and add me back as a reviewer. Also, you'll want to use clang-format to fix spacing.