Skip to content

Commit 2b92b95

Browse files
committed
Fix: more struggles with broken build
1 parent eb4513a commit 2b92b95

6 files changed

Lines changed: 14 additions & 8 deletions

File tree

src/Core/Configuration/PlatformDefinition.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ public virtual IPlatform Load(IServiceProvider services, IProcessorArchitecture
115115
var type = svc.GetType(TypeName);
116116
if (type is null)
117117
throw new TypeLoadException(
118-
string.Format("Unable to load {0} environment.", Description));
118+
$"Unable to load {Description} environment.");
119119
var platform = (Platform)Activator.CreateInstance(type, services, arch)!;
120120
LoadSettingsFromConfiguration(services, platform);
121121
return platform;

src/Environments/SysV/ArchSpecific/ArchSpecificFactory.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,7 @@ public ArchSpecificFactory(IServiceProvider services, IProcessorArchitecture arc
130130
var t = svc.GetType("Reko.Arch.X86.X86CallingConvention,Reko.Arch.X86");
131131
this.ccX86 = (ICallingConvention)Activator.CreateInstance(
132132
t,
133+
arch,
133134
4, // stackAlignment,
134135
4, // pointerSize,
135136
true, // callerCleanup,

src/Native/ArmNative/Arm64Rewriter.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ class Arm64Rewriter : public ComBase, public INativeRewriter
4545
STDMETHOD_(ULONG, AddRef)() override { return ComBase::AddRef(); }
4646
STDMETHOD_(ULONG, Release)() override { return ComBase::Release(); }
4747

48-
STDMETHOD_(int32_t, Next)();
49-
STDMETHOD_(int32_t, GetCount)();
48+
STDMETHOD_(int32_t, Next)() override;
49+
STDMETHOD_(int32_t, GetCount)() override;
5050

5151
private:
5252
void AddConditional(void(*mkInstr)());

src/Native/ArmNative/ThumbArchitecture.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ class ThumbArchitecture : public ComBase, public INativeArchitecture
2121
public:
2222
ThumbArchitecture();
2323

24-
virtual STDMETHODIMP QueryInterface(REFIID riid, void ** ppvObject);
25-
virtual STDMETHODIMP_(ULONG) AddRef(void) override { return ComBase::AddRef(); }
26-
virtual STDMETHODIMP_(ULONG) Release(void) override { return ComBase::Release(); }
24+
STDMETHODIMP QueryInterface(REFIID riid, void ** ppvObject) override;
25+
STDMETHODIMP_(ULONG) AddRef(void) override { return ComBase::AddRef(); }
26+
STDMETHODIMP_(ULONG) Release(void) override { return ComBase::Release(); }
2727

2828
STDMETHODIMP GetAllRegisters(int category, int * pcRegs, void * * ppRegs) override;
2929
STDMETHODIMP_(INativeDisassembler *) CreateDisassembler(void * bytes, int length, int offset, uint64_t uAddr) override;

src/UnitTests/Arch/X86/Disassembler/V20DisassemblerTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ public void V20Dis_repnc_for_cmps_scas()
111111
public void V20Dis_rep_kept_for_non_compare_string_instructions()
112112
{
113113
var repMovs = Disassemble(v20Arch, LoadAddress, "F3 A4");
114-
Assert.AreEqual("rep movsb", repMovs.ToString());
114+
Assert.AreEqual("rep movbk", repMovs.ToString());
115115
}
116116

117117
// V20 bit-manipulation instructions (0F prefix)

src/tools/c2xml/Program.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,8 +113,13 @@ public int DoExecute(string [] args)
113113
catch (Exception ex)
114114
{
115115
Console.WriteLine(
116-
$"c2xml: an error occurred when loading platform {envElem.Name} ({envElem.TypeName}). {ex.Message}");
116+
$"c2xml: an error occurred when loading platform {envElem.Name} ({envElem.TypeName}).");
117+
for (Exception e = ex; e is not null; e = e.InnerException)
118+
{
119+
Console.WriteLine($" {e.Message}");
120+
}
117121
Console.WriteLine(ex.StackTrace);
122+
Console.WriteLine("Constructor(s):");
118123
return 1;
119124
}
120125

0 commit comments

Comments
 (0)