commit 268764da8022cafa5661c5b514eaa343c5257e57
author Mathieu Chartier <mathieuc@google.com> Tue Sep 13 12:09:38 2016 -0700
committer Mathieu Chartier <mathieuc@google.com> Tue Sep 13 15:18:29 2016 -0700
tree 7c4b46847a6dcfac7a1492edd79b434fb1a0d6ea
parent f9decf915f6fa5b33fceba7c10c119e4cad481e5

Make ScopedAssertNoThreadSuspension no overhead for non-debug

Previously it required Thread::Current() which may not be free.
The plan is to add a lot more ScopedAssertNoThreadSuspension in
the codebase.

Also cleaned up callers.

Bug: 31458474
Change-Id: I5a1621a5435476504d22266cc01a9bf26aab7568

compiler/driver/compiler_driver.cc

diff --git a/compiler/driver/compiler_driver.cc b/compiler/driver/compiler_driver.cc
index daac7fb..72c54fe 100644
--- a/compiler/driver/compiler_driver.cc
+++ b/compiler/driver/compiler_driver.cc
@@ -2473,7 +2473,7 @@
               // mode which prevents the GC from visiting objects modified during the transaction.
               // Ensure GC is not run so don't access freed objects when aborting transaction.
 
-              ScopedAssertNoThreadSuspension ants(soa.Self(), "Transaction end");
+              ScopedAssertNoThreadSuspension ants("Transaction end");
               runtime->ExitTransactionMode();
 
               if (!success) {