Merge pull request #281 from kevinbackhouse/GlobalValueNumberingBugFix

Workaround for getFullyConverted returning multiple results.

Author: rdmarsh2

cpp/ql/src/semmle/code/cpp/valuenumbering/GlobalValueNumbering.qll

@@ -312,7 +312,7 @@ private predicate analyzableLocalScopeVariable(VariableAccess access) {
   strictcount (SsaDefinition def, Variable v | def.getAUse(v) = access | v) = 1 and
   count (SsaDefinition def, Variable v
   | def.getAUse(v) = access
-  | def.getDefiningValue(v)) <= 1 and
+  | def.getDefiningValue(v).getFullyConverted()) <= 1 and
   not analyzableConst(access)
 }