[PATCH] D13659: Implement .reloc (constant offset only) with support for R_MIPS_NONE and R_MIPS_32.
Daniel Sanders via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 27 13:02:50 PDT 2015
dsanders updated this revision to Diff 38584.
dsanders added a comment.
Fix all review comments except for the addend hook which I'm still thinking
about:
- Loc argument no longer optional.
- Fixed accidental success/fail inversion in MCStreamer::EmitRelocDirective. I've retained true==failure despite my misgivings about it since it's consistent with the other Emit*() functions.
- .reloc expression is now optional.
http://reviews.llvm.org/D13659
Files:
include/llvm/MC/MCAsmBackend.h
include/llvm/MC/MCObjectStreamer.h
include/llvm/MC/MCStreamer.h
lib/MC/MCAsmBackend.cpp
lib/MC/MCAsmStreamer.cpp
lib/MC/MCObjectStreamer.cpp
lib/MC/MCParser/AsmParser.cpp
lib/Target/Mips/MCTargetDesc/MipsAsmBackend.cpp
lib/Target/Mips/MCTargetDesc/MipsAsmBackend.h
lib/Target/Mips/MCTargetDesc/MipsELFObjectWriter.cpp
lib/Target/Mips/MCTargetDesc/MipsFixupKinds.h
test/MC/Mips/reloc-directive.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13659.38584.patch
Type: text/x-patch
Size: 13403 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151027/8133eb78/attachment.bin>
More information about the llvm-commits
mailing list