First Last Prev Next    No search results available
Details
: [X86] X86 Backend never releases memory for machine code ...
Bug#: 192
: libraries
: Backend: X86
Status: RESOLVED
Resolution: FIXED
: All
: All
: 1.0
: P2
: normal
: 1.2

:
: quality-of-implementation
:
:
  Show dependency tree - Show dependency graph
People
Reporter: Chris Lattner <sabre@nondot.org>
Assigned To: Chris Lattner <sabre@nondot.org>

Attachments


Note

You need to log in before you can comment on or make changes to this bug.

Related actions


Description:   Opened: 2003-12-20 04:22
The X86 backend never frees memory for MachineInstr, MachineBasicBlock,
MachineFunction, etc objects, even after it has emitted the function or whatever
ot memory or the .s file.

This is silly.

-Chris
------- Comment #1 From Chris Lattner 2003-12-20 04:27:16 -------
This is now fixed.  Here are the patches:
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20031215/010390.html
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20031215/010392.html
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20031215/010393.html

While I was at it, I fixed this minor memory leak.  I wish MachineInstr's used
ilists.  :(
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20031215/010389.html

Also, this permits removing this FIXME from the JIT:
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20031215/010391.html

It appears that the Sparc code generator is not freeing MachineFunction objects,
but is freeing most of the other gunk.  I've emailed Misha a proposed fix for test.

-Chris
------- Comment #2 From Misha Brukman 2003-12-21 21:50:04 -------
Misha has tested Chris' suggested fix and committed it:

http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20031215/010408.html
------- Comment #3 From Chris Lattner 2003-12-21 21:55:09 -------
Thanks Misha!

-Chris

First Last Prev Next    No search results available