diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2014-12-11 21:39:39 +0000 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2014-12-11 21:39:39 +0000 |
| commit | 63eb6bf6233dded6c91ac50bd2ec76e054aadba3 (patch) | |
| tree | a9a7349a44cf1c3491ade43d09a0714d23573850 /llvm/lib/CodeGen/MachineBasicBlock.cpp | |
| parent | 7e37a5f52347f7bc9c09471e731e3209e0583a3d (diff) | |
| download | bcm5719-llvm-63eb6bf6233dded6c91ac50bd2ec76e054aadba3.tar.gz bcm5719-llvm-63eb6bf6233dded6c91ac50bd2ec76e054aadba3.zip | |
IR: Store MDNodes in a separate LeakDetector container
This gives us better leak detection messages, like `Value` has.
This also has the side effect of papering over a problem where
`MachineInstr`s are added as garbage to the leak detector and then
deleted without being removed. If `MDNode::getTemporary()` allocates an
`MDNodeFwdDecl` in the same spot, the leak detector asserts. By
separating `MDNode`s into their own container we lose that assertion.
Since `MachineInstr` is required to have a trivial destructor, its usage
of `LeakDetector` at all is pretty suspect. I'll be sending a patch
soon to strip that out.
llvm-svn: 224060
Diffstat (limited to 'llvm/lib/CodeGen/MachineBasicBlock.cpp')
0 files changed, 0 insertions, 0 deletions
