[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