[PATCH] D33194: [DWARF] - Cleanup relocations proccessing.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 17 01:48:38 PDT 2017


grimar added a comment.

Thanks for review, Adrian !

I am planning to commit this after https://reviews.llvm.org/D33184 probably, because
we have RelocAddrEntry now:

  struct RelocAddrEntry {
    uint8_t Width;
    int64_t Value;
  };
  typedef DenseMap<uint64_t, RelocAddrEntry> RelocAddrMap;

And if I remove Width field, like this patch do, then struct with single field
would look unreasonable. https://reviews.llvm.org/D33184 adds one more field and should look fine.


https://reviews.llvm.org/D33194





More information about the llvm-commits mailing list