summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Trommler <ptrommler@acm.org>2021-07-18 17:28:48 +0200
committerBen Gamari <ben@well-typed.com>2021-07-28 15:51:13 +0000
commit1b89584ce66195da74f97652f8f4d6fd4ab286ff (patch)
treefa1be41e80ff71d7429f80b4575dac07229d17ff
parent10678945c1d3261273a1d7a389d14a69f4e28567 (diff)
downloadhaskell-wip/T20130.tar.gz
Delete ToDo about incorrect optimisation [skip ci]wip/T20130
On big-endian systems a narrow after a load cannot be replaced with a narrow load.
-rw-r--r--compiler/GHC/Cmm/Opt.hs4
1 files changed, 0 insertions, 4 deletions
diff --git a/compiler/GHC/Cmm/Opt.hs b/compiler/GHC/Cmm/Opt.hs
index 8035d7b2bd..676c1270b5 100644
--- a/compiler/GHC/Cmm/Opt.hs
+++ b/compiler/GHC/Cmm/Opt.hs
@@ -108,10 +108,6 @@ cmmMachOpFoldM platform conv_outer [CmmMachOp conv_inner [x]]
intconv True = MO_SS_Conv
intconv False = MO_UU_Conv
--- ToDo: a narrow of a load can be collapsed into a narrow load, right?
--- but what if the architecture only supports word-sized loads, should
--- we do the transformation anyway?
-
cmmMachOpFoldM platform mop [CmmLit (CmmInt x xrep), CmmLit (CmmInt y _)]
= case mop of
-- for comparisons: don't forget to narrow the arguments before