Delete the copy constructor for llvm::yaml::Node
authorJordan Rose <jordan_rose@apple.com>
Fri, 23 Mar 2018 18:05:19 +0000 (18:05 +0000)
committerJordan Rose <jordan_rose@apple.com>
Fri, 23 Mar 2018 18:05:19 +0000 (18:05 +0000)
commitdc86de6b6e3f34c669296f4b4fbc87075c41cc8e
treecd4df1914575f98962ab28dc00b6df6af0da0600
parent80f10e4fe5987cda01724ba9663b6d96754e3169
Delete the copy constructor for llvm::yaml::Node

The nodes keep a reference back to the original document, but the
document is streamed, not read all into memory at once, and the
position is part of the state. If nodes are ever copied, the document
position can end up being advanced more than once.

This did not reveal any problems in LLVM or Clang but caught a handful
over in Swift!

llvm-svn: 328345
llvm/include/llvm/Support/YAMLParser.h