Skip to content

Combine exprIsTrivial and cpe_ExprIsTrivial

Look at CoreUtils.exprIsTrivial and CorePrep.cpe_ExprIsTrivial. They are identical!

The latter has a comment saying

cpe_ExprIsTrivial :: CoreExpr -> Bool
-- Version that doesn't consider an scc annotation to be trivial.

but the code does not treat ticks any differently.

So the question is: are they supposed to be different? And if not, can we just combine them?

Peter W, Simon M: this is your territory. I don't understand tick stuff well enough.

Simon

Trac metadata
Trac field Value
Version 7.10.2
Type Bug
TypeOfFailure OtherFailure
Priority normal
Resolution Unresolved
Component Compiler
Test case
Differential revisions
BlockedBy
Related
Blocking
CC
Operating system
Architecture
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information