diff options
author | Jakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com> | 2021-07-01 22:39:04 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-07-01 22:39:04 +0300 |
commit | b8beed27f441acbad86fca4a37f7e1287fec071f (patch) | |
tree | ced3708f05a23cac790e0e0d475a8cb3d036bfff /src/coreclr/jit/flowgraph.cpp | |
parent | 09785ef78f8c758350ec1ded604484bbbed0ce99 (diff) |
Poison address-exposed user variables in debug (#54685)
* Poison address exposed user variables in debug code
Fix #13072
* Run jit-format
* Use named scratch register and kill it in LSRA
* Enable it unconditionally for testing purposes
* Remove unnecessary modified reg on ARM
* Fix OSR and get rid of test code
* Remove a declaration
* Undo modified comment and use modulo instead of and
* Add a test
* Rephrase comment
Co-authored-by: Kunal Pathak <Kunal.Pathak@microsoft.com>
* Disable poisoning test on mono
* Remove outdated line
Co-authored-by: Kunal Pathak <Kunal.Pathak@microsoft.com>
Diffstat (limited to 'src/coreclr/jit/flowgraph.cpp')
-rw-r--r-- | src/coreclr/jit/flowgraph.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/coreclr/jit/flowgraph.cpp b/src/coreclr/jit/flowgraph.cpp index e7a281cdeca..04faed33f50 100644 --- a/src/coreclr/jit/flowgraph.cpp +++ b/src/coreclr/jit/flowgraph.cpp @@ -2603,8 +2603,8 @@ void Compiler::fgAddInternal() noway_assert(!compIsForInlining()); // The backend requires a scratch BB into which it can safely insert a P/Invoke method prolog if one is - // required. Create it here. - if (compMethodRequiresPInvokeFrame()) + // required. Similarly, we need a scratch BB for poisoning. Create it here. + if (compMethodRequiresPInvokeFrame() || compShouldPoisonFrame()) { fgEnsureFirstBBisScratch(); fgFirstBB->bbFlags |= BBF_DONT_REMOVE; |