[PATCH] D28516: [ELF] - Explicitly list supported relocations for x86 target.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 10 12:15:31 PST 2017


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: ELF/Target.cpp:387
+  default:
+    error("do not know how to handle relocation " + toString(Type) + "(" +
+          Twine(Type) + ")");
----------------
Put a space before '('.


https://reviews.llvm.org/D28516





More information about the llvm-commits mailing list