Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ODIN II: Instantiating more than one module with dual_port_ram inside fails #2

Closed
kmurray opened this issue Jun 26, 2015 · 5 comments
Closed
Assignees
Labels
bug Incorrect behaviour Odin Odin II Logic Synthesis Tool: Unsorted item

Comments

@kmurray
Copy link
Contributor

kmurray commented Jun 26, 2015

Originally reported on Google Code with ID 9

What steps will reproduce the problem?
1. ./odin_II.exe -V test2.v -a sample_arch.xml
2.
3.

What is the expected output? What do you see instead?
Instantiating more than one instance of a module, which contains a dual_port_ram inside,
should work.
Instead, ODIN II errors out with "Missing declaration of this symbol dual_port_ram"

Workaround is to copy-and-paste the module in question with another name. See attached
for example.

What version of the product are you using? On what operating system?
ODIN II version 0.1 (from VTR1.0rc1) under 64-bit Linux


Please provide any additional information below.


Reported by eddie.hung on 2012-01-20 01:01:23


- _Attachment: [test2.v](https://storage.googleapis.com/google-code-attachments/vtr-verilog-to-routing/issue-9/comment-0/test2.v)_
@kmurray kmurray self-assigned this Jun 26, 2015
@kmurray
Copy link
Contributor Author

kmurray commented Jun 26, 2015

Oops. Missing architecture file (taken from VPR directory)

Reported by eddie.hung on 2012-01-20 01:02:03


- _Attachment: [sample_arch.xml](https://storage.googleapis.com/google-code-attachments/vtr-verilog-to-routing/issue-9/comment-1/sample_arch.xml)_

@kmurray
Copy link
Contributor Author

kmurray commented Jun 26, 2015

Reported by jeffrey.goeders on 2012-01-20 18:17:44

  • Labels added: Module-ODIN

@kmurray
Copy link
Contributor Author

kmurray commented Jun 26, 2015

Thank you for discovering this issue. 

The problem stems from the fact that Odin II currently fails to prefix hard blocks
with the module instance name and simply uses the module name. The original author
of this code must have neglected to include the module instance name when naming the
hard blocks. 

I've got someone looking into it, so it should be fixed soon. 

Reported by andy16666 on 2012-04-18 12:17:13

@kmurray kmurray added bug Incorrect behaviour Odin Odin II Logic Synthesis Tool: Unsorted item and removed Module-ODIN labels Sep 15, 2016
@kmurray kmurray assigned KennethKent and unassigned kmurray Jan 2, 2017
@jeanlego jeanlego self-assigned this Jul 16, 2017
kmurray added a commit that referenced this issue Apr 2, 2018
1fc200f Merge pull request #7 from sterin/master
9c78efb Makefile: add support for ABC_USE_STDINT_H
4a39f32 Merge pull request #2 from rqou/master
8d472cd Rename new flag to ABC_USE_STDINT_H
d879336 Merge pull request #5 from sterin/master
c5aebf6 README: minor updates
2fd7ba5 Merge pull request #1 from gpshead/patch-1
41eb4ea Merge pull request #3 from kmurray/fix_cmake_libabc_dependency
c3be5dc CMake: Ensure abc executable depends on libabc
40c8a39 Add an option to use C99 stdint.h
ce4b3cf point to github instead of bitbucket

git-subtree-dir: abc
git-subtree-split: 1fc200ffacabed1796639b562181051614f5fedb
vaughnbetz referenced this issue in rfungquicklogic/vtr-verilog-to-routing Feb 8, 2019
…ity for adjacent pins (likely to be in the same equivalence groups).
@mflawn
Copy link
Contributor

mflawn commented Jun 6, 2019

Issue unresolved

Attached file and sample_arch.xml to reproduce
test2.v.txt

./odin_II -V test2.v -a ../libs/libarchfpga/arch/sample_arch.xml  

	Optimizing module by AST based optimizations 
	Converting AST into a Netlist. Note this netlist can be viewed using GraphViz (see documentation)
	==========================
	Detected Top Level Module:      m
	==========================
	-------------- 
	Odin has decided you have failed ;)

	ERROR (1):NETLIST_ERROR (File: test2.v) (Line number: 41) Missing declaration of this symbol dual_port_ram
			  .addr2(addr2));
	ASSERT FAILED:
			@[/mnt/c/Users/casa/vtr-verilog-to-routing/ODIN_II/SRC/ast_util.cpp]get_name_of_pins::813
	Aborted (core dumped)

@jeanlego
Copy link
Contributor

fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect behaviour Odin Odin II Logic Synthesis Tool: Unsorted item
Projects
None yet
Development

No branches or pull requests

5 participants