Skip to content

Commit

Permalink
[tests] make Symbols.FunctionNames aware of interpreter (#4973)
Browse files Browse the repository at this point in the history
  • Loading branch information
lewurm authored and spouliot committed Oct 16, 2018
1 parent 13b2647 commit 60b06b9
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion tests/monotouch-test/mono/Symbols.cs
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,13 @@ public void FunctionNames ()
Assert.Ignore ("This is a device-only test.");

Collect ();
Assert.That (symbols [1], Contains.Substring ("MonoTouchFixtures_Symbols_Collect"), "#1");
bool aot = symbols [1].Contains ("MonoTouchFixtures_Symbols_Collect");
/* ves_pinvoke_method (slow path) and do_icall (fast path) are
* MONO_NEVER_INLINE, so they should show up in the backtrace
* reliably */
bool interp = symbols [1].Contains ("ves_pinvoke_method") || symbols [1].Contains ("do_icall");

Assert.IsTrue (aot || interp, "#1");
}

void Collect ()
Expand Down

1 comment on commit 60b06b9

@xamarin-release-manager
Copy link
Collaborator

Choose a reason for hiding this comment

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

Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Build' 🔥 : hudson.AbortException: script returned exit code 2

🔥 Build failed 🔥

Please sign in to comment.