[llvm-commits] [llvm] r168536 - in /llvm/trunk: include/llvm/MC/MCDwarf.h lib/MC/MCDwarf.cpp

[Sean Silva]

On Fri, Nov 23, 2012 at 11:33 PM, Rafael Espindola
<rafael.espindola at gmail.com> wrote:
>      static MCCFIInstruction createDefCfaOffset(MCSymbol *L, int Offset) {
> -      MachineLocation Dest(MachineLocation::VirtualFP);
> -      MachineLocation Source(MachineLocation::VirtualFP, -Offset);
> -      MCCFIInstruction Ret(OpDefCfaOffset, L, Dest, Source, "");
> +      MCCFIInstruction Ret(OpDefCfaOffset, L, 0, -Offset, "");
>        return Ret;

For this and other places in the patch, could we just

return MCCFIInstruction(OpDefCfaOffset, L, 0, -Offset, "");

?

-- Sean Silva