av_fetch(): optimise the negative index branch.

For a negative index one conditional is redundant, since after determining
that key < 0 and recomputing key as (AvFILLp(av) - key), key can't
be > AvFILLp(av).

1 files changed, 4 insertions, 2 deletions

diff --git a/av.c b/av.c
index a48702bb7d..8967da5703 100644
--- a/av.c
+++ b/av.c
@@ -272,9 +272,11 @@ Perl_av_fetch(pTHX_ AV *av, SSize_t key, I32 lval)
 	key += AvFILLp(av) + 1;
 	if (key < 0)
 	    return NULL;
+        assert(key <= AvFILLp(av));
+        if (!AvARRAY(av)[key])
+            goto emptyness;
     }
-
-    if (key > AvFILLp(av) || !AvARRAY(av)[key]) {
+    else if (key > AvFILLp(av) || !AvARRAY(av)[key]) {
       emptyness:
 	return lval ? av_store(av,key,newSV(0)) : NULL;
     }