[PATCH] cpp11-migrate: Add '-supports' command line switch.

Sean Silva silvas at purdue.edu
Wed Jul 24 13:05:35 PDT 2013


  FYI, when I read your description of the option, it seemed like what it did was choose transforms that have been available in cpp11-migrate since clang-3.0 (which is completely different from what the flag actually does).

  The `-supports` flag is not very self-documenting; can you think of a name that would be self-documenting? TBH I can't think of a good, short name for this; maybe `-filter-transforms-for-compilers=clang-3.0,gcc-4.8,icc-12.1`? (I'm not very happy with that name though)


================
Comment at: docs/MigratorUsage.rst:121-123
@@ +120,5 @@
+
+  There is a special platform **c++11** that enables all transforms.
+
+  ``cpp11-migrate -supports c++11 <args..>``
+
----------------
Is there ever a situation where `-supports c++11` would alter behavior? If not, then why have it?

================
Comment at: docs/MigratorUsage.rst:116-119
@@ -115,1 +115,6 @@
 
+.. option:: -supports <platform>
+
+  Select the platforms to support. Transforms will be selected automatically to
+  work on all selected platforms.
+
----------------
This description is not very clear. How about "Select transforms targeting the intersection of language features supported by the given compilers".


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

BRANCH
  supports-cmd-2

ARCANIST PROJECT
  clang-tools-extra



More information about the cfe-commits mailing list