Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/32927 )

Change subject: x86: Replace getDoubleBits with floatToBits64.
......................................................................

x86: Replace getDoubleBits with floatToBits64.

The getDoubleBits function was used exactly once to find the bit
representation of a double floating point value, which is the same thing
the common floatToBits64 function does. Eliminate x86's one off version,
and use the common one instead.

Change-Id: Icb0cec5a55d81a6eacf1bb5a3c2b8f16c414d0d9
---
M src/arch/x86/isa/microops/limmop.isa
M src/arch/x86/utility.hh
2 files changed, 1 insertion(+), 8 deletions(-)



diff --git a/src/arch/x86/isa/microops/limmop.isa b/src/arch/x86/isa/microops/limmop.isa
index 1fdbcf4..9333c00 100644
--- a/src/arch/x86/isa/microops/limmop.isa
+++ b/src/arch/x86/isa/microops/limmop.isa
@@ -144,7 +144,7 @@
             if isinstance(imm, (int, long)):
                 imm = "ULL(%d)" % imm
             elif isinstance(imm, float):
-                imm = "getDoubleBits(%.16f)" % imm
+                imm = "floatToBits64(%.16f)" % imm
             self.imm = imm
             self.dataSize = dataSize

diff --git a/src/arch/x86/utility.hh b/src/arch/x86/utility.hh
index b311eef..39a142c 100644
--- a/src/arch/x86/utility.hh
+++ b/src/arch/x86/utility.hh
@@ -114,13 +114,6 @@
     void setRFlags(ThreadContext *tc, uint64_t val);

     /**
-     * Extract the bit string representing a double value.
-     */
-    inline uint64_t getDoubleBits(double val) {
-        return *(uint64_t *)(&val);
-    }
-
-    /**
      * Convert an x87 tag word to abridged tag format.
      *
      * Convert from the x87 tag representation to the tag abridged

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/32927
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Icb0cec5a55d81a6eacf1bb5a3c2b8f16c414d0d9
Gerrit-Change-Number: 32927
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <gabebl...@google.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to