This is an archive of the discontinued LLVM Phabricator instance.

MipsAsmParser: Fix a couple of memory leaks
ClosedPublic

Authored by chapuni on Apr 15 2014, 6:40 AM.

Details

Reviewers
dsanders
Summary
  1. Manage Mem.Base.
  2. Be responsible to release temporary Operands in MipsAsmParser::ParseRegister().

See also; http://lab.llvm.org:8011/builders/llvm-x86_64-linux-vg_leak
Feel free to rewrite and commit one as yourselves.

Diff Detail

Event Timeline

dsanders accepted this revision.Apr 15 2014, 7:06 AM

LGTM. Thanks for looking into this.

chapuni closed this revision.Apr 15 2014, 7:21 AM

Applied in r206292 and r206293, thanks!