This patch fixes trivial mishandling of 64-bit instructions that may potentially cause verification errors with -verify-machineinstrs.
This patch should not bring functional changes.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Aside from the inline stylistic nit, this LGTM.
lib/Target/PowerPC/PPCRegisterInfo.cpp | ||
---|---|---|
378 | Just a stylistic nit:
In fact, I'd probably define something like unsigned FramePtrReg = LP64 ? PPC::X31 : PPC::R31 just after line 375 and just use it (along with a ternary expression for the opcode) in this call to BuildMI(). |