Displaying 2 results from an estimated 2 matches for "mcdwarfframeinfo".
2013 Dec 19
1
[LLVMdev] [PATCH] MC: handle .cfi_startproc simple
...MC/MCAsmStreamer.cpp
> index 781e400..9fb3c5d 100644
> --- a/lib/MC/MCAsmStreamer.cpp
> +++ b/lib/MC/MCAsmStreamer.cpp
> @@ -61,7 +61,7 @@ private:
> bool needsSet(const MCExpr *Value);
>
> void EmitRegisterName(int64_t Register);
> - virtual void EmitCFIStartProcImpl(MCDwarfFrameInfo &Frame);
> + virtual void EmitCFIStartProcImpl(MCDwarfFrameInfo &Frame, bool Simple);
> virtual void EmitCFIEndProcImpl(MCDwarfFrameInfo &Frame);
>
> public:
> @@ -902,13 +902,15 @@ void MCAsmStreamer::EmitCFISections(bool EH, bool Debug) {
> EmitEOL();
> }...
2013 Jul 14
0
[LLVMdev] [PATCH v2] MC: handle .cfi_startproc simple
Hi,
After your change, EmitCFIStartProcImpl in MCAsmStreamer does not
match the signature of the EmitCFIStartProcImpl in MCStreamer and you
end up not overriding the original function. One of the places where
"virtual void EmitCFIStartProcImpl(MCDwarfFrameInfo &Frame) = 0" would
have helped. :)
Specifically, adding
virtual void EmitCFIStartProcImpl(MCDwarfFrameInfo &Frame) {
EmitCFIStartProcImpl(Frame, false);
}
to MCAsmStreamer gets llvm to pass all tests.
I'm not familiar with this part of llvm, but my guess about the
right...