member descriptor ics

trying to fill in rewriting holes... for this one I had to some support for doubles and some `mov` variants in the assembler and rewriter. Kind of hacky/lazy...
well the byte turns into an int so I had to extend it, but the bool is just
in the lower 8 bits

On Mon, Sep 22, 2014 at 8:34 PM, Kevin Modzelewski <notifications@github.com
> wrote:

> In src/runtime/objmodel.cpp:
>
> >                  int8_t rtn = reinterpret_cast<int8_t*>(obj)[member_desc->offset];
> >                  return boxInt(rtn);
> >              }
> >              case BoxedMemberDescriptor::BOOL: {
> > -                rewrite_args = NULL;
> > +                if (rewrite_args) {
> > +                    RewriterVarUsage r_unboxed_val
> > +                        = rewrite_args->obj.getAttr(member_desc->offset, RewriterVarUsage::KillFlag::Kill,
> > +                                                    Location::forArg(0), assembler::MovType::B);
>
> Hmm I feel iffy about this one -- why does it need a different MovType
> than for BYTE?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dropbox/pyston/pull/151/files#r17890893>.
>