Rule 20.12: Improve performance

This commit optimizes the from-where-select clause of this query, which
was highlighted as one of the slowest predicates for our C query suites.

This commit makes the following changes:
 * Avoid repetition of `m.getParameter(i)`, which caused a cross product
 * Change the way we identify "further expanded macro"
 * Modify the isExcluded predicate to reference the macro invocation not
   the macro.

Author: lcartey

c/misra/src/rules/RULE-20-12/MacroParameterUsedAsHashOperand.ql

@@ -19,7 +19,7 @@ import codingstandards.cpp.Macro
 
 from FunctionLikeMacro m, MacroInvocation mi, int i, string expanded, string param
 where
-  not isExcluded(m, Preprocessor2Package::macroParameterUsedAsHashOperandQuery()) and
+  not isExcluded(mi, Preprocessor2Package::macroParameterUsedAsHashOperandQuery()) and
   mi = m.getAnInvocation() and
   param = m.getParameter(i) and
   (
@@ -31,9 +31,6 @@ where
   // This check ensure there is an expansion that is used.
   expanded = mi.getExpandedArgument(i) and
   not expanded = "" and
-  exists(Macro furtherExpandedMacro |
-    mi.getUnexpandedArgument(i).matches(furtherExpandedMacro.getName() + "%")
-  )
+  not mi.getUnexpandedArgument(i) = mi.getExpandedArgument(i)
 select m,
-  "Macro " + m.getName() + " contains use of parameter " + m.getParameter(i) +
-    " used in multiple contexts."
+  "Macro " + m.getName() + " contains use of parameter " + param + " used in multiple contexts."