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

Remove native strict math testing code. #77

Merged
merged 1 commit into from
Sep 21, 2017

Conversation

hangshao0
Copy link
Contributor

Native implementations of the following strict match APIs have already
been remove:

Java_java_lang_StrictMath_cos()
Java_java_lang_StrictMath_acos()
Java_java_lang_StrictMath_log()
Java_java_lang_StrictMath_sin()
Java_java_lang_StrictMath_asin()
Java_java_lang_StrictMath_pow()
Java_java_lang_StrictMath_tan()
Java_java_lang_StrictMath_atan()
Java_java_lang_StrictMath_atan2()
Java_java_lang_StrictMath_exp()

The corresponding testing code is obsolete, and should be removed.

Signed-off-by: Hang Shao hangshao@ca.ibm.com

Native implementations of the following strict match APIs have already
been remove:

Java_java_lang_StrictMath_cos()
Java_java_lang_StrictMath_acos()
Java_java_lang_StrictMath_log()
Java_java_lang_StrictMath_sin()
Java_java_lang_StrictMath_asin()
Java_java_lang_StrictMath_pow()
Java_java_lang_StrictMath_tan()
Java_java_lang_StrictMath_atan()
Java_java_lang_StrictMath_atan2()
Java_java_lang_StrictMath_exp()

The corresponding testing code is obsolete, and should be removed.

Signed-off-by: Hang Shao <hangshao@ca.ibm.com>
@pshipton
Copy link
Member

It seems the java_lang_Math tests are obsolete as well, and the entire flsanity suite should be removed.

@hangshao0
Copy link
Contributor Author

Yes. The entire flsanity suite can be removed.
I also noticed some other obsolete code outside the /flsanity directory. It will be a bigger change, so I will remove all the non-strictmath related code through another pull request.

@genie-openj9
Copy link

Can one of the admins verify this patch?

@pshipton pshipton merged commit cdd70c8 into eclipse-openj9:master Sep 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants