Skip to content

Commit

Permalink
[Test] Remove uniques from 'Profiling' tests (#6483)
Browse files Browse the repository at this point in the history
This removes uniques from some golden test files, because those add a lot of noise in unrelated PRs.
  • Loading branch information
effectfully authored Sep 14, 2024
1 parent 179e07c commit e34cda9
Show file tree
Hide file tree
Showing 15 changed files with 197 additions and 197 deletions.
2 changes: 1 addition & 1 deletion plutus-tx-plugin/src/PlutusTx/Compiler/Expr.hs
Original file line number Diff line number Diff line change
Expand Up @@ -524,7 +524,7 @@ maybeProfileRhs var t = do
CompileContext{ccOpts = compileOpts} <- ask
let ty = PLC._varDeclType var
varName = PLC._varDeclName var
displayName = T.pack $ PP.displayPlc varName
displayName = T.pack $ PP.displayPlcSimple varName
isFunctionOrAbstraction = case ty of PLC.TyFun{} -> True; PLC.TyForall{} -> True; _ -> False
-- Trace only if profiling is on *and* the thing being defined is a function
if coProfile compileOpts == All && isFunctionOrAbstraction
Expand Down
8 changes: 4 additions & 4 deletions plutus-tx-plugin/test/Plugin/Profiling/9.6/addInt.pir.golden
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ program
in
trace
{unit -> integer}
"entering addInteger-131"
"entering addInteger"
(\(thunk : unit) ->
trace {integer} "exiting addInteger-131" (addInteger x y))
trace {integer} "exiting addInteger" (addInteger x y))
()
~addInt : integer -> integer -> integer
= \(x : integer) ->
Expand All @@ -24,9 +24,9 @@ program
in
trace
{unit -> integer -> integer}
"entering addInt-128"
"entering addInt"
(\(thunk : unit) ->
trace {integer -> integer} "exiting addInt-128" (addInteger x))
trace {integer -> integer} "exiting addInt" (addInteger x))
()
in
addInt)
Original file line number Diff line number Diff line change
@@ -1 +1 @@
[entering addInt-128, exiting addInt-128]
[entering addInt, exiting addInt]
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[ entering runIdentity-131
, exiting runIdentity-131
, entering newtypeFunction-139
, exiting newtypeFunction-139
, entering `$fFoldableIdentity`-133
, exiting `$fFoldableIdentity`-133 ]
[ entering runIdentity
, exiting runIdentity
, entering newtypeFunction
, exiting newtypeFunction
, entering `$fFoldableIdentity`
, exiting `$fFoldableIdentity` ]
Original file line number Diff line number Diff line change
@@ -1 +1 @@
[entering obscuredFunction-129, exiting obscuredFunction-129]
[entering obscuredFunction, exiting obscuredFunction]
72 changes: 36 additions & 36 deletions plutus-tx-plugin/test/Plugin/Profiling/9.6/fact4.eval.golden
Original file line number Diff line number Diff line change
@@ -1,36 +1,36 @@
[ entering fact-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-152
, exiting subtractInteger-152
, entering fact-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-152
, exiting subtractInteger-152
, entering fact-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-152
, exiting subtractInteger-152
, entering fact-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-152
, exiting subtractInteger-152
, entering fact-128
, entering equalsInteger-135
, exiting equalsInteger-135
, exiting fact-128
, entering multiplyInteger-146
, exiting multiplyInteger-146
, exiting fact-128
, entering multiplyInteger-146
, exiting multiplyInteger-146
, exiting fact-128
, entering multiplyInteger-146
, exiting multiplyInteger-146
, exiting fact-128
, entering multiplyInteger-146
, exiting multiplyInteger-146
, exiting fact-128 ]
[ entering fact
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fact
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fact
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fact
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fact
, entering equalsInteger
, exiting equalsInteger
, exiting fact
, entering multiplyInteger
, exiting multiplyInteger
, exiting fact
, entering multiplyInteger
, exiting multiplyInteger
, exiting fact
, entering multiplyInteger
, exiting multiplyInteger
, exiting fact
, entering multiplyInteger
, exiting multiplyInteger
, exiting fact ]
16 changes: 8 additions & 8 deletions plutus-tx-plugin/test/Plugin/Profiling/9.6/fib.pir.golden
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ program
in
trace
{unit -> integer}
"entering addInteger-150"
"entering addInteger"
(\(thunk : unit) ->
trace {integer} "exiting addInteger-150" (addInteger x y))
trace {integer} "exiting addInteger" (addInteger x y))
()
data Bool | Bool_match where
True : Bool
Expand All @@ -33,11 +33,11 @@ program
in
trace
{unit -> Bool}
"entering equalsInteger-135"
"entering equalsInteger"
(\(thunk : unit) ->
trace
{Bool}
"exiting equalsInteger-135"
"exiting equalsInteger"
(let
!b : bool = equalsInteger x y
in
Expand All @@ -55,11 +55,11 @@ program
in
trace
{unit -> integer}
"entering subtractInteger-156"
"entering subtractInteger"
(\(thunk : unit) ->
trace
{integer}
"exiting subtractInteger-156"
"exiting subtractInteger"
(subtractInteger x y))
()
in
Expand All @@ -71,11 +71,11 @@ program
in
trace
{unit -> integer}
"entering fib-128"
"entering fib"
(\(thunk : unit) ->
trace
{integer}
"exiting fib-128"
"exiting fib"
(Bool_match
(equalsInteger n 0)
{all dead. integer}
Expand Down
148 changes: 74 additions & 74 deletions plutus-tx-plugin/test/Plugin/Profiling/9.6/fib4.eval.golden
Original file line number Diff line number Diff line change
@@ -1,74 +1,74 @@
[ entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering equalsInteger-135
, exiting equalsInteger-135
, exiting fib-128
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, exiting fib-128
, entering addInteger-150
, exiting addInteger-150
, exiting fib-128
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering equalsInteger-135
, exiting equalsInteger-135
, exiting fib-128
, entering addInteger-150
, exiting addInteger-150
, exiting fib-128
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering equalsInteger-135
, exiting equalsInteger-135
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, entering equalsInteger-135
, exiting equalsInteger-135
, exiting fib-128
, entering subtractInteger-156
, exiting subtractInteger-156
, entering fib-128
, entering equalsInteger-135
, exiting equalsInteger-135
, exiting fib-128
, entering addInteger-150
, exiting addInteger-150
, exiting fib-128
, entering addInteger-150
, exiting addInteger-150
, exiting fib-128 ]
[ entering fib
, entering equalsInteger
, exiting equalsInteger
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, entering equalsInteger
, exiting equalsInteger
, exiting fib
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, exiting fib
, entering addInteger
, exiting addInteger
, exiting fib
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, entering equalsInteger
, exiting equalsInteger
, exiting fib
, entering addInteger
, exiting addInteger
, exiting fib
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, entering equalsInteger
, exiting equalsInteger
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, entering equalsInteger
, exiting equalsInteger
, exiting fib
, entering subtractInteger
, exiting subtractInteger
, entering fib
, entering equalsInteger
, exiting equalsInteger
, exiting fib
, entering addInteger
, exiting addInteger
, exiting fib
, entering addInteger
, exiting addInteger
, exiting fib ]
2 changes: 1 addition & 1 deletion plutus-tx-plugin/test/Plugin/Profiling/9.6/id.eval.golden
Original file line number Diff line number Diff line change
@@ -1 +1 @@
[entering id-129, exiting id-129, entering id-129, exiting id-129]
[entering id, exiting id, entering id, exiting id]
4 changes: 2 additions & 2 deletions plutus-tx-plugin/test/Plugin/Profiling/9.6/idCode.pir.golden
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ program
\(x : a) ->
trace
{unit -> a}
"entering id-129"
(\(thunk : unit) -> trace {a} "exiting id-129" x)
"entering id"
(\(thunk : unit) -> trace {a} "exiting id" x)
()
in
id {integer} (id {integer} 1))
20 changes: 10 additions & 10 deletions plutus-tx-plugin/test/Plugin/Profiling/9.6/letInFun.eval.golden
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
[ entering f-140
, entering addInteger-134
, exiting addInteger-134
, exiting f-140
, entering f-140
, entering addInteger-134
, exiting addInteger-134
, exiting f-140
, entering addInteger-134
, exiting addInteger-134 ]
[ entering f
, entering addInteger
, exiting addInteger
, exiting f
, entering f
, entering addInteger
, exiting addInteger
, exiting f
, entering addInteger
, exiting addInteger ]
Loading

1 comment on commit e34cda9

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Plutus Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.05.

Benchmark suite Current: e34cda9 Previous: 179e07c Ratio
validation-decode-game-sm-success_2-6 169.4 μs 160.6 μs 1.05

This comment was automatically generated by workflow using github-action-benchmark.

CC: @IntersectMBO/plutus-core

Please sign in to comment.