FIXME comments for Ares64

src/BizHawk.Emulation.Cores/Consoles/Nintendo/Ares64/Ares64.IDebuggable.cs

@@ -21,6 +21,8 @@ namespace BizHawk.Emulation.Cores.Consoles.Nintendo.Ares64
 			ret.Add("LO", data[32]);
 			ret.Add("HI", data[33]);
 			ret.Add("PC", (uint)data[34]);
+			// FIXME: the PC register is actually 64-bits (although in practice it will only ever be 32-bits)
+			// Debugger UI doesn't like it as 64 bits, hence the uint cast
 
 			return ret;
 		}

src/BizHawk.Emulation.Cores/Consoles/Nintendo/Ares64/Ares64.IDisassemblable.cs

@@ -22,6 +22,7 @@ namespace BizHawk.Emulation.Cores.Consoles.Nintendo.Ares64
 
 		public override string Disassemble(MemoryDomain m, uint addr, out int length)
 		{
+			// FIXME: the PC register is actually 64-bits (although in practice it will only ever be 32-bits)
 			_core.GetDisassembly(addr, m.PeekUint(addr, true), _disasmbuf);
 			length = 4;
 			var ret = Encoding.UTF8.GetString(_disasmbuf);

src/BizHawk.Emulation.Cores/Consoles/Nintendo/Ares64/Ares64.ITraceable.cs

@@ -13,6 +13,8 @@ namespace BizHawk.Emulation.Cores.Consoles.Nintendo.Ares64
 
 		private void MakeTrace(IntPtr disasm)
 		{
+			// FIXME: Something is wrong here, this is called back once and never called back again
+			// worked fine before v128 merge, probably some upstream regression?
 			var disasmStr = Mershul.PtrToStringUtf8(disasm);
 			if (!disasmStr.StartsWith("CPU")) // garbage, ignore
 			{