[PATCH] cpp11-migrate: Add a transform that use pass-by-value in constructors

Guillaume Papin guillaume.papin at epitech.eu
Tue Aug 27 13:29:23 PDT 2013


  Two changes since the last patch to fix build issues:

  - In cpp11-migrate/PassByValue/PassByValue.cpp, LLVM_OVERRIDE has been removed
    from the out-of-line definition of the PassByValueTransform::apply() method.

  - Use -fno-delayed-template-parsing in the tests (see the top comments in
    test/cpp11-migrate/PassByValue/basic.cpp) and add "Note about delayed template
    parsing" in the RST documentation of the transform.

Hi revane, arielbernal, tareqsiraj,

http://llvm-reviews.chandlerc.com/D1342

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D1342?vs=3812&id=3827#toc

BRANCH
  pass-by-value

ARCANIST PROJECT
  clang-tools-extra

Files:
  cpp11-migrate/Core/Transform.h
  cpp11-migrate/PassByValue/PassByValue.cpp
  cpp11-migrate/PassByValue/PassByValue.h
  cpp11-migrate/PassByValue/PassByValueActions.cpp
  cpp11-migrate/PassByValue/PassByValueActions.h
  cpp11-migrate/PassByValue/PassByValueMatchers.cpp
  cpp11-migrate/PassByValue/PassByValueMatchers.h
  cpp11-migrate/tool/CMakeLists.txt
  cpp11-migrate/tool/Cpp11Migrate.cpp
  cpp11-migrate/tool/Makefile
  docs/MigratorUsage.rst
  docs/PassByValueTransform.rst
  docs/cpp11-migrate.rst
  test/cpp11-migrate/PassByValue/basic.cpp
  test/cpp11-migrate/PassByValue/basic.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1342.6.patch
Type: text/x-patch
Size: 34266 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130827/dc91b608/attachment.bin>


More information about the cfe-commits mailing list