[PATCH] D20688: [ELF][MIPS] Handle section symbol points to the .MIPS.options / .reginfo section

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu May 26 11:04:08 PDT 2016


ruiu added inline comments.

================
Comment at: ELF/InputSection.cpp:73-74
@@ +72,4 @@
+    // section. Redirect it to the produced output section.
+    assert(Offset == 0 &&
+           "Retrieve offset in the middle of MIPS reginfo/options section");
+    return this->OutSec->getVA();
----------------
Please use fatal instead of assert because this can be triggered by a broken user input. (use assert only for programmer's (i.e. our) errors.)

================
Comment at: test/ELF/mips-options-r.test:12
@@ +11,3 @@
+#   nop
+# % as -mabi=64 -mips64r2 t.s
+
----------------
Can't you create it using llvm-as?


Repository:
  rL LLVM

http://reviews.llvm.org/D20688





More information about the llvm-commits mailing list