Fix a memory leak in the hexagon scheduler. We call initialize here more
authorChandler Carruth <chandlerc@gmail.com>
Sat, 27 Jul 2013 10:48:45 +0000 (10:48 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Sat, 27 Jul 2013 10:48:45 +0000 (10:48 +0000)
commit37886438dd9e8b734f1a090eb2ede48a4d184f0e
tree58607ca49796aecca8f3055bf3e124baeb07f39a
parent89934cbd344b6839f4d298b281998a9f48355bf1
Fix a memory leak in the hexagon scheduler. We call initialize here more
than once, and the second time through we leaked memory. Found thanks to
the vg-leak bot, but I can't locally reproduce it with valgrind. The
debugger confirms that it is in fact leaking here.

This whole code is totally gross. Why is initialize being called on each
runOnFunction??? Why aren't these OwningPtr<>s, and why aren't their
lifetimes better defined? Anyways, this is just a surgical change to
help out the leak checking bots.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@187299 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/Hexagon/HexagonMachineScheduler.cpp