From: Argyrios Kyrtzidis <akyrtzi@gmail.com>
Date: Wed, 3 Apr 2013 21:29:07 +0000 (+0000)
Subject: [preprocessor] Minor optimization following r178671.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4ad33d0f971da3eba5041f13fbd034f80ea9bfeb;p=clang

[preprocessor] Minor optimization following r178671.

Don't bother looking for parameter index of 'B' token if 'A' is not a parameter.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@178699 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Lex/MacroInfo.cpp b/lib/Lex/MacroInfo.cpp
index 5abafe1cb7..b61ff71d17 100644
--- a/lib/Lex/MacroInfo.cpp
+++ b/lib/Lex/MacroInfo.cpp
@@ -110,8 +110,9 @@ bool MacroInfo::isIdenticalTo(const MacroInfo &Other, Preprocessor &PP,
       // With syntactic equivalence the parameter names can be different as long
       // as they are used in the same place.
       int AArgNum = getArgumentNum(A.getIdentifierInfo());
-      int BArgNum = Other.getArgumentNum(B.getIdentifierInfo());
-      if (AArgNum == -1 || AArgNum != BArgNum)
+      if (AArgNum == -1)
+        return false;
+      if (AArgNum != Other.getArgumentNum(B.getIdentifierInfo()))
         return false;
       continue;
     }