Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

ci: test two GHDL backends (mcode and LLVM) #43

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

umarcor
Copy link
Contributor

@umarcor umarcor commented May 24, 2021

Instead of testing compilation with GHDL's mcode backend only, this PR adds another job for testing with GHDL's LLVM backend too. However, GHDL LLVM chokes on the usage of pin_bit_information in libraries ibm and clib. I asked Tristan about it in ghdl/ghdl#1772.

The last commit in this PR comments all those attributes so that GHDL can analyse/elaborate them. Yet, Tristan commented:

ghdl/ghdl#1772 (comment)

But in fact the aggregate is not valid. The type is an array of an array of a string, but the length of the strings is not the same.

This is not the reason of the crash, but even if the feature were handled, the source won't be analyzed successfully.

Therefore, I will mark this PR as draft, for discussion. If those attributes are used 200+ times in this codebase, I would expect them to be supported in whichever internal VHDL simulator/synthesiser used in IBM.

@openpowerwtf
Copy link
Collaborator

The repo has moved to https://github.com/OpenPOWERFoundation/a2i.

I referenced this PR in discussion: OpenPOWERFoundation/a2i#2.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants