commit 62a319071ae9243f63609546bc23d899e73ba6ec
author njn <njn@a5019735-40e9-0310-863c-91ae7b9d1cf9> Sun Aug 28 05:21:17 2005 +0000
committer njn <njn@a5019735-40e9-0310-863c-91ae7b9d1cf9> Sun Aug 28 05:21:17 2005 +0000
tree 4209d5490a53d49d02c4461d75e9647cc9b44a8a
parent d9ab15df5e000a1393fa6ef6e2f7481eef89eabe

Add some comments.



git-svn-id: svn://svn.valgrind.org/valgrind/trunk@4548 a5019735-40e9-0310-863c-91ae7b9d1cf9

addrcheck/ac_main.c

diff --git a/addrcheck/ac_main.c b/addrcheck/ac_main.c
index db05274..221440e 100644
--- a/addrcheck/ac_main.c
+++ b/addrcheck/ac_main.c
@@ -707,6 +707,7 @@
    abits >>= (a & 4);
    abits &= 15;
    PROF_EVENT(66, "");
+   // XXX: Should this be "if (!is_distinguished_sm(sm) && abits == VGM_NIBBLE_VALID)"?
    if (abits == VGM_NIBBLE_VALID) {
       /* Handle common case quickly: a is suitably aligned, is mapped,
          and is addressible.  So just return. */
@@ -727,6 +728,7 @@
    AcSecMap* sm     = primary_map[sec_no];
    UInt    a_off  = (SM_OFF(a)) >> 3;
    PROF_EVENT(67, "");
+   // XXX: Should this be "if (!is_distinguished_sm(sm) && sm->abits[a_off] == VGM_BYTE_VALID)"?
    if (sm->abits[a_off] == VGM_BYTE_VALID) {
       /* Handle common case quickly. */
       return;
@@ -746,6 +748,7 @@
    AcSecMap* sm   = primary_map[sec_no];
    UInt    a_off  = (SM_OFF(a)) >> 3;
    PROF_EVENT(68, "");
+   // XXX: Should this be "if (!is_distinguished_sm(sm) && sm->abits[a_off] == VGM_BYTE_VALID)"?
    if (sm->abits[a_off] == VGM_BYTE_VALID) {
       /* Handle common case quickly. */
       return;