summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Target/ARM/Thumb2ITBlockPass.cpp
diff options
context:
space:
mode:
authorChandler Carruth <chandlerc@gmail.com>2014-03-10 01:32:25 +0000
committerChandler Carruth <chandlerc@gmail.com>2014-03-10 01:32:25 +0000
commitca7fbae3f6a4df35755b05eaf0bb7890fb435778 (patch)
tree4b4febb755c143c6169e419f5e2dc496df9cd42d /llvm/lib/Target/ARM/Thumb2ITBlockPass.cpp
parent596587f6d0bf932f4a97100790a2d75d41250b1c (diff)
downloadbcm5719-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
OpenPOWER on IntegriCloud