diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-01-12 19:22:04 +0000 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-01-12 19:22:04 +0000 |
| commit | 5f4618923cce10f1b2c3e7e646a986d3bcf43a73 (patch) | |
| tree | 017631c6c219b4baea17c4c77449fec2b85f8e3d /llvm/lib | |
| parent | 967629e14abc76ecf8f6516ae6fd6b4a0069170a (diff) | |
| download | bcm5719-llvm-5f4618923cce10f1b2c3e7e646a986d3bcf43a73.tar.gz bcm5719-llvm-5f4618923cce10f1b2c3e7e646a986d3bcf43a73.zip | |
IR: Add test for handleChangedOperand() recursion
Turns out this can happen. Remove the `FIXME` and add a testcase that
crashes without the extra logic.
llvm-svn: 225657
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/IR/Metadata.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/llvm/lib/IR/Metadata.cpp b/llvm/lib/IR/Metadata.cpp index 1ae9a96fe46..baa0b78762c 100644 --- a/llvm/lib/IR/Metadata.cpp +++ b/llvm/lib/IR/Metadata.cpp @@ -537,8 +537,6 @@ void GenericMDNode::handleChangedOperand(void *Ref, Metadata *New) { if (InRAUW) { // We just hit a recursion due to RAUW. Set the operand and move on, since // we're about to be deleted. - // - // FIXME: Can this cycle really happen? setOperand(Op, New); return; } |
