[PATCH] D31875: [AVR] Migrate to new MCAsmBackend applyFixup
Leslie Zhai via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 10 19:19:07 PDT 2017
xiangzhai added a comment.
Hi Jonathan,
Thanks for your review!
> Why didn't the lack of this break the build?
In file included from /data/project/LLVM/llvm/lib/Target/AVR/MCTargetDesc/AVRAsmBackend.cpp:14:
/data/project/LLVM/llvm/lib/Target/AVR/MCTargetDesc/AVRAsmBackend.h:44:55: error: non-virtual member function marked 'override' hides virtual member function
uint64_t Value, bool IsPCRel) const override;
^
/data/project/LLVM/llvm/include/llvm/MC/MCAsmBackend.h:76:16: note: hidden overloaded virtual function 'llvm::MCAsmBackend::applyFixup' declared here: different number of parameters (6 vs 5)
virtual void applyFixup(const MCFixup &Fixup, char *Data, unsigned DataSize,
^
/data/project/LLVM/llvm/lib/Target/AVR/MCTargetDesc/AVRAsmBackend.cpp:469:14: error: allocating an object of abstract class type 'llvm::AVRAsmBackend'
return new AVRAsmBackend(TT.getOS());
^
/data/project/LLVM/llvm/include/llvm/MC/MCAsmBackend.h:76:16: note: unimplemented pure virtual method 'applyFixup' in 'AVRAsmBackend'
virtual void applyFixup(const MCFixup &Fixup, char *Data, unsigned DataSize,
^
2 errors generated.
And I do not have commit access, please commit it on my behalf, thanks a lot!
Regards,
Leslie Zhai
Repository:
rL LLVM
https://reviews.llvm.org/D31875
More information about the llvm-commits
mailing list