emit >>0 in HEAP8, for consistency with HEAP16|32

Author: kripken

src/parseTools.js

@@ -1168,15 +1168,10 @@ function getHeapOffset(offset, type, forceAsm) {
   var sz = Runtime.getNativeTypeSize(type);
   var shifts = Math.log(sz)/Math.LN2;
   offset = '(' + offset + ')';
-  if (shifts != 0) {
-    if (CHECK_HEAP_ALIGN) {
-      return '((CHECK_ALIGN_' + sz + '(' + offset + '|0)|0)>>' + shifts + ')';
-    } else {
-      return '(' + offset + '>>' + shifts + ')';
-    }
+  if (CHECK_HEAP_ALIGN && shifts > 0) {
+    return '((CHECK_ALIGN_' + sz + '(' + offset + '|0)|0)>>' + shifts + ')';
   } else {
-    // we need to guard against overflows here, HEAP[U]8 expects a guaranteed int
-    return isJSVar(offset) ? offset : '(' + offset + '|0)';
+    return '(' + offset + '>>' + shifts + ')';
   }
 }