[PATCH] Extend loop variable naming checks

[Jack Yang]

  As per suggestions, the fix for this issue has been completely reimplemented. The original variable naming logic will still exist, but the checks for conflicts has been extended to check for language keywords (including aliases), macros, and types.

  New tests have been added. Also renamed naming.cpp to naming-alias.cpp but I guess git diff treats it as a new file instead of a rename.

Hi klimek,

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

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D484?vs=1165&id=1201#toc

Files:
  cpp11-migrate/LoopConvert/LoopActions.cpp
  cpp11-migrate/LoopConvert/StmtAncestor.cpp
  cpp11-migrate/LoopConvert/StmtAncestor.h
  cpp11-migrate/LoopConvert/VariableNaming.cpp
  cpp11-migrate/LoopConvert/VariableNaming.h
  test/cpp11-migrate/LoopConvert/naming-alias.cpp
  test/cpp11-migrate/LoopConvert/naming-conflict.cpp
  test/cpp11-migrate/LoopConvert/naming.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D484.2.patch
Type: text/x-patch
Size: 11575 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130306/a8d65b5a/attachment.bin>