Re: [build] Release build is broken

2007-11-28 Thread Aleksey Shipilev
Fixed, no build problems.

On Nov 28, 2007 5:52 PM, Mikhail Fursov <[EMAIL PROTECTED]> wrote:
> Please check r599017
>
>
> On Nov 28, 2007 8:38 PM, Mikhail Fursov <[EMAIL PROTECTED]> wrote:
> > I'll check and commit if it's still not fixed.
> >
> >
> > On Nov 28, 2007 8:35 PM, Aleksey Shipilev <[EMAIL PROTECTED]> wrote:
> > > Hi all,
> > >
> > > I have tried to build r599011 on Windows/ia32 and failed with this error:
> > >
> > > == cut ==
> > >626  build.native.init:
> > >627   [echo] ## Building native of 'vm.jitrino'
> > >628
> > >629  build.native.c:
> > >630 [cc] 0 total files to be compiled.
> > >631
> > >632  build.native.cpp:
> > >633 [cc] Starting dependency analysis for 151 files.
> > >634 [cc] 151 files are up to date.
> > >635 [cc] 0 files to be recompiled from dependency analysis.
> > >636 [cc] 1 total files to be compiled.
> > >[cc] cl : Command line warning D4025 : overriding '/W3' with '/W4'
> > >637 [cc] rt.cpp
> > >638 [cc]
> > > C:\users\shade\Harmony\trunk\working_vm\vm\jitrino\src\jet\rt.cpp(407)
> > > : error C2065: 'meth_start' :
> > >  undeclared identifier
> > >
> > > BUILD FAILED
> > > C:\users\shade\Harmony\trunk\working_vm\build\make\build.xml:581: The
> > > following error occurred while executing this line
> > > :
> > > C:\users\shade\Harmony\trunk\working_vm\build\make\build.xml:588: The
> > > following error occurred while executing this line
> > > :
> > > C:\users\shade\Harmony\trunk\working_vm\build\make\build_component.xml:74:
> > > The following error occurred while executing
> > > this line:
> > > C:\users\shade\Harmony\trunk\working_vm\build\make\targets\build.native.xml:107:
> > > cl failed with return code 2
> > >
> > > Total time: 18 seconds
> > > == cut ==
> > >
> > > I had a quick look into the sources and my minor change:
> > >
> > > Index: vm/jitrino/src/jet/rt.cpp
> > > ===
> > > --- vm/jitrino/src/jet/rt.cpp   (revision 599011)
> > > +++ vm/jitrino/src/jet/rt.cpp   (working copy)
> > > @@ -397,8 +397,9 @@
> > >
> > >  void *** pip = (void***)((char*)context + ip_off);
> > >  char * where = (char*)**pip;
> > > +
> > > +char * meth_start = infoBlock.get_code_start();
> > >  #ifdef _DEBUG
> > > -char * meth_start = infoBlock.get_code_start();
> > >  unsigned meth_len = infoBlock.get_code_len();
> > >  assert(meth_start <= where);
> > >  assert(where < meth_start + meth_len);
> > >
> > > ...solves the problem. Could JIT gurus review this fix and commit it?
> > >
> > > Thanks,
> > > Aleksey,
> > > ESSD, Intel.
> > >
> >
> >
> >
> > --
> > Mikhail Fursov
> >
>
>
>
> --
> Mikhail Fursov
>


Re: [build] Release build is broken

2007-11-28 Thread Mikhail Fursov
Please check r599017

On Nov 28, 2007 8:38 PM, Mikhail Fursov <[EMAIL PROTECTED]> wrote:
> I'll check and commit if it's still not fixed.
>
>
> On Nov 28, 2007 8:35 PM, Aleksey Shipilev <[EMAIL PROTECTED]> wrote:
> > Hi all,
> >
> > I have tried to build r599011 on Windows/ia32 and failed with this error:
> >
> > == cut ==
> >626  build.native.init:
> >627   [echo] ## Building native of 'vm.jitrino'
> >628
> >629  build.native.c:
> >630 [cc] 0 total files to be compiled.
> >631
> >632  build.native.cpp:
> >633 [cc] Starting dependency analysis for 151 files.
> >634 [cc] 151 files are up to date.
> >635 [cc] 0 files to be recompiled from dependency analysis.
> >636 [cc] 1 total files to be compiled.
> >[cc] cl : Command line warning D4025 : overriding '/W3' with '/W4'
> >637 [cc] rt.cpp
> >638 [cc]
> > C:\users\shade\Harmony\trunk\working_vm\vm\jitrino\src\jet\rt.cpp(407)
> > : error C2065: 'meth_start' :
> >  undeclared identifier
> >
> > BUILD FAILED
> > C:\users\shade\Harmony\trunk\working_vm\build\make\build.xml:581: The
> > following error occurred while executing this line
> > :
> > C:\users\shade\Harmony\trunk\working_vm\build\make\build.xml:588: The
> > following error occurred while executing this line
> > :
> > C:\users\shade\Harmony\trunk\working_vm\build\make\build_component.xml:74:
> > The following error occurred while executing
> > this line:
> > C:\users\shade\Harmony\trunk\working_vm\build\make\targets\build.native.xml:107:
> > cl failed with return code 2
> >
> > Total time: 18 seconds
> > == cut ==
> >
> > I had a quick look into the sources and my minor change:
> >
> > Index: vm/jitrino/src/jet/rt.cpp
> > ===
> > --- vm/jitrino/src/jet/rt.cpp   (revision 599011)
> > +++ vm/jitrino/src/jet/rt.cpp   (working copy)
> > @@ -397,8 +397,9 @@
> >
> >  void *** pip = (void***)((char*)context + ip_off);
> >  char * where = (char*)**pip;
> > +
> > +char * meth_start = infoBlock.get_code_start();
> >  #ifdef _DEBUG
> > -char * meth_start = infoBlock.get_code_start();
> >  unsigned meth_len = infoBlock.get_code_len();
> >  assert(meth_start <= where);
> >  assert(where < meth_start + meth_len);
> >
> > ...solves the problem. Could JIT gurus review this fix and commit it?
> >
> > Thanks,
> > Aleksey,
> > ESSD, Intel.
> >
>
>
>
> --
> Mikhail Fursov
>



-- 
Mikhail Fursov


Re: [build] Release build is broken

2007-11-28 Thread Mikhail Fursov
I'll check and commit if it's still not fixed.

On Nov 28, 2007 8:35 PM, Aleksey Shipilev <[EMAIL PROTECTED]> wrote:
> Hi all,
>
> I have tried to build r599011 on Windows/ia32 and failed with this error:
>
> == cut ==
>626  build.native.init:
>627   [echo] ## Building native of 'vm.jitrino'
>628
>629  build.native.c:
>630 [cc] 0 total files to be compiled.
>631
>632  build.native.cpp:
>633 [cc] Starting dependency analysis for 151 files.
>634 [cc] 151 files are up to date.
>635 [cc] 0 files to be recompiled from dependency analysis.
>636 [cc] 1 total files to be compiled.
>[cc] cl : Command line warning D4025 : overriding '/W3' with '/W4'
>637 [cc] rt.cpp
>638 [cc]
> C:\users\shade\Harmony\trunk\working_vm\vm\jitrino\src\jet\rt.cpp(407)
> : error C2065: 'meth_start' :
>  undeclared identifier
>
> BUILD FAILED
> C:\users\shade\Harmony\trunk\working_vm\build\make\build.xml:581: The
> following error occurred while executing this line
> :
> C:\users\shade\Harmony\trunk\working_vm\build\make\build.xml:588: The
> following error occurred while executing this line
> :
> C:\users\shade\Harmony\trunk\working_vm\build\make\build_component.xml:74:
> The following error occurred while executing
> this line:
> C:\users\shade\Harmony\trunk\working_vm\build\make\targets\build.native.xml:107:
> cl failed with return code 2
>
> Total time: 18 seconds
> == cut ==
>
> I had a quick look into the sources and my minor change:
>
> Index: vm/jitrino/src/jet/rt.cpp
> ===
> --- vm/jitrino/src/jet/rt.cpp   (revision 599011)
> +++ vm/jitrino/src/jet/rt.cpp   (working copy)
> @@ -397,8 +397,9 @@
>
>  void *** pip = (void***)((char*)context + ip_off);
>  char * where = (char*)**pip;
> +
> +char * meth_start = infoBlock.get_code_start();
>  #ifdef _DEBUG
> -char * meth_start = infoBlock.get_code_start();
>  unsigned meth_len = infoBlock.get_code_len();
>  assert(meth_start <= where);
>  assert(where < meth_start + meth_len);
>
> ...solves the problem. Could JIT gurus review this fix and commit it?
>
> Thanks,
> Aleksey,
> ESSD, Intel.
>



-- 
Mikhail Fursov