MIPT-ILab / MDSP

[2009 – 2012] MDSP: functional simulation of a Multimedia Digital Signal Processor
MIT License
3 stars 1 forks source link

Fix regular testing detected problem [FAIL] MDSP testing results 03.12.2010 12:23.05 #36

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

r469 introduced a problem that leads to the fact that Makefile build of MDSP 
fails.

This issue is critical and must be fixed ASAP.

Original issue reported on code.google.com by grigory....@gmail.com on 3 Dec 2010 at 10:07

GoogleCodeExporter commented 9 years ago

Original comment by grigory....@gmail.com on 3 Dec 2010 at 12:08

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
I got a proof that r469 is the first buggy one. If you check it out and try to 
build it will fail with linking:

<... checkout ...>
Checked out revision 469.

C:\Users\grechist\mdsp-tests>cd r469

C:\Users\grechist\mdsp-tests\r469>make
<...skipped...>

g++ -Wall -O3 Release/build/funcsim/config.o Release/build/funcsim/core.o Releas
e/build/funcsim/flags.o Release/build/funcsim/log.o Release/build/funcsim/memory
.o Release/build/funcsim/operation.o Release/build/funcsim/register_file.o Relea
se/build/funcsim/main.o -o Release/funcsim
Release/build/funcsim/main.o:main.cpp:(.text+0x79): undefined reference to `Sche
duler::init(int, char**)'
collect2: ld returned 1 exit status
make: *** [funcsim] Error 1

The previous main trunk revision r464 does not suffer from this bug:

C:\Users\grechist\mdsp-tests\r469>svn update -r 464
D    sources/funcsim/main_functions.cpp
D    sources/funcsim/scheduler.cpp
D    sources/funcsim/scheduler.h
U    sources/funcsim/main_functions.h
U    sources/funcsim/main.cpp
U    sources/funcsim/core.cpp
U    sources/funcsim/core.h
 U   .
Updated to revision 464.

C:\Users\grechist\mdsp-tests\r469>make clean
rm -rf Release/build
rm -f Release/funcsim
rm -f Release/test
rm -f Release/asm

C:\Users\grechist\mdsp-tests\r469>make
<...skipped...>

So, the owner is set up correctly.

Original comment by grigory....@gmail.com on 3 Dec 2010 at 2:50

GoogleCodeExporter commented 9 years ago

Original comment by vladimir...@gmail.com on 3 Dec 2010 at 3:45