Update Append Argument to more efficiently traverse tokens
authorErich Keane <erich.keane@intel.com>
Thu, 15 Jun 2017 00:27:23 +0000 (00:27 +0000)
committerErich Keane <erich.keane@intel.com>
Thu, 15 Jun 2017 00:27:23 +0000 (00:27 +0000)
commit5b767760e9dd5d92c17604ccf425300fe4cb2d45
tree8d1eb4bc78a728f2958cf13cc5a334ac3ce450a9
parent7901b4709455a98096e436221d21760ab68a55ad
Update Append Argument to more efficiently traverse tokens

This function was previously making (correct) assumptions
without complete knowledge of MacroArgs guarantees for
Arguments.  After going through Macro Args a bunch, I'd
corrected the getNumArguments (and changed its name),
however didn't realize this was depending on the behavior.

This patch has version that depends on the corrected
getNumMacroArguments's behavior, with the rest checked against
my knowledge of the MacroArgs' token list.  Commiting no-wait
since the test is broken.

llvm-svn: 305434
clang-tools-extra/pp-trace/PPCallbacksTracker.cpp