Avoid populating shared patch bypass buffer after initial creation (#117514)

* Avoid populating shared patch bypass buffer after initial creation

* Assert m_pSharedPatchBypassBuffer is NULL in CreateSharedPatchBypassBuffer

* Centralize shared bypass patch buffer creation

* Move debugger lock to DebuggerPatchSkip ctor

* Update comment in controller.cpp

Co-authored-by: Noah Falk <noahfalk@users.noreply.github.com>

* Refactor shared patch buffer handling in DebuggerControllerPatch.
Encapsulate instruction attribute into SharedBypasBuffer

---------

Co-authored-by: Noah Falk <noahfalk@users.noreply.github.com>

Author: tommcdon

src/coreclr/debug/daccess/stdafx.h

@@ -42,6 +42,7 @@
 #include <common.h>
 #include <codeman.h>
 #include <debugger.h>
+#include <walker.h>
 #include <controller.h>
 #include <eedbginterfaceimpl.h>
 #include <methoditer.h>