-
Notifications
You must be signed in to change notification settings - Fork 11k
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
[flang] Run real 10 test on x86 only (NFC) #73911
Conversation
@llvm/pr-subscribers-flang-fir-hlfir @llvm/pr-subscribers-flang-semantics Author: None (madanial0) ChangesRemove real 10 tests for powerpc Full diff: https://github.com/llvm/llvm-project/pull/73911.diff 1 Files Affected:
diff --git a/flang/unittests/Evaluate/real.cpp b/flang/unittests/Evaluate/real.cpp
index 9701082898388be..6c37c28d913c7ff 100644
--- a/flang/unittests/Evaluate/real.cpp
+++ b/flang/unittests/Evaluate/real.cpp
@@ -14,7 +14,9 @@ using Real2 = Scalar<Type<TypeCategory::Real, 2>>;
using Real3 = Scalar<Type<TypeCategory::Real, 3>>;
using Real4 = Scalar<Type<TypeCategory::Real, 4>>;
using Real8 = Scalar<Type<TypeCategory::Real, 8>>;
+#ifndef __PPC__
using Real10 = Scalar<Type<TypeCategory::Real, 10>>;
+#endif
using Real16 = Scalar<Type<TypeCategory::Real, 16>>;
using Integer4 = Scalar<Type<TypeCategory::Integer, 4>>;
using Integer8 = Scalar<Type<TypeCategory::Integer, 8>>;
@@ -538,7 +540,9 @@ void roundTest(int rm, Rounding rounding, std::uint32_t opds) {
basicTests<Real3>(rm, rounding);
basicTests<Real4>(rm, rounding);
basicTests<Real8>(rm, rounding);
+ #ifndef __PPC__
basicTests<Real10>(rm, rounding);
+ #endif
basicTests<Real16>(rm, rounding);
ScopedHostFloatingPointEnvironment::SetRounding(rounding);
subsetTests<std::uint32_t, float, Real4>(rm, rounding, opds);
|
✅ With the latest revision this PR passed the C/C++ code formatter. |
I don't really contribute to this area, so removing myself from the list of reviewers. |
It is not supported on AArch64 as well. |
Would you like me to add AArch64 macro to that test case as well? |
Yes, that would be great. Does it fail currently or give a warning? |
Added changes to |
The macro approach does not seem to be working in the fortran files resulting in a failure will remove the |
Which macro are you using? It might be that the flang driver is not defining them properly. |
The test is using |
I am using |
I tried the above still did not work with the |
The macros are added in the following location.
|
It seems the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG.
This reverts commit 943a6cbd4d7c6ada6341d68401e46108e6c0b83b.
Remove real 10 tests for powerpc