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

[Rafael Espíndola]

> Agreed, I was looking at it earlier and couldn't figure out what was meant
> in 30 seconds or so. Some comments would be great :)

This is part of the old representation that is really confusing. For a
MachineLocation, isReg means "it is *only* a register". For example,
isReg is false for a Reg,Offset pair:

  MachineLocation(unsigned R, int O)
    : IsRegister(false), Register(R), Offset(O) {}

I want to delete MachineLocation completely and implement
getInitialFrameState directly with MCCFIInstruction, but to do that I
need to get EH out of the legacy JIT first.

> -eric

Cheers,
Rafael