RE: Status of tests

Project Infrastructure LLVM Dev List Archives
1

What's the current status of the test suite. After putting everything
in the llvm namespace and getting everything to link, I am getting=20
only about 75% of the tests passing. Is this to be expected or have
I broken something?

I think that you broke something, specifically the C backend. See this
comment for details:
http://llvm.cs.uiuc.edu/bugs/show_bug.cgi?id=109#c12

Basically you accidentally renamed 'llvm-dis -c' to 'llvm-dis -c_code'.

-Chris

2

Okay, I’ll fix that. But something WILL have to change, its not gratuitious.

The “-c” option is in an enum which also has a “llvm” value which conflicts with the namespace name and caused a compilation error.
To avoid the error, I changed “llvm” to “llvm_code”. To be consistent, I changed the “c” enumeration to also have the same suffix, “_code”.

I wasn’t aware when changing this that I’d be changing the command line arguments.

I’ll work up another solution.

REid.

<i>> What's the current status of the test suite.  After putting everything
> in the llvm namespace and getting everything to link, I am getting=20
> only about 75% of the tests passing. Is this to be expected or have
> I broken something?

I think that you broke something, specifically the C backend.  See this
comment for details:</i><i>
[http://llvm.cs.uiuc.edu/bugs/show_bug.cgi?id=109#c12](http://llvm.cs.uiuc.edu/bugs/show_bug.cgi?id=109#c12)

Basically you accidentally renamed 'llvm-dis -c' to 'llvm-dis -c_code'.

-Chris</i>


> <br><br><br><br><br><br> |
> - |
3

Yup, I understand it needs to change. That's what the clEnumValN thing
is for :slight_smile:

-Chris