diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2014-03-10 01:32:25 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2014-03-10 01:32:25 +0000 |
commit | ca7fbae3f6a4df35755b05eaf0bb7890fb435778 (patch) | |
tree | 4b4febb755c143c6169e419f5e2dc496df9cd42d /llvm/lib/Target/ARM/Thumb2ITBlockPass.cpp | |
parent | 596587f6d0bf932f4a97100790a2d75d41250b1c (diff) | |
download | bcm5719-llvm-ca7fbae3f6a4df35755b05eaf0bb7890fb435778.tar.gz bcm5719-llvm-ca7fbae3f6a4df35755b05eaf0bb7890fb435778.zip |
[PM] As Dave noticed in review, I had erroneously copied the move
constructors from the classes which only have a single reference member
to many other places. This resulted in them copying their single member
instead of moving. =/ Fix this.
There's really not a useful test to add sadly because these move
constructors are only called when something deep inside some standard
library implementation *needs* to move them. Many of the types aren't
even user-impacting types. Or, the objects are copyable anyways and so
the result was merely a performance problem rather than a correctness
problem.
Anyways, thanks for the review. And this is a great example of why
I wish I colud have the compiler write these for me.
llvm-svn: 203431
Diffstat (limited to 'llvm/lib/Target/ARM/Thumb2ITBlockPass.cpp')
0 files changed, 0 insertions, 0 deletions