commit 6275f2083415d22a6ce0de55645079cd47e0cc80
author Dmitriy Ivanov <dimitry@google.com> Mon May 12 11:36:56 2014 -0700
committer Dmitriy Ivanov <dimitry@google.com> Mon May 12 11:47:10 2014 -0700
tree 15d2210e61586d0d2c2ef41b0b176b08531f8f19
parent e29979503a166d51e18e557a16aa8703a0dbdbb9

Cleanup: updated comments

Bug: 14566672
Change-Id: I49ad28e3914b7d559db1e98542fee919db768f49

linker/linker.cpp

diff --git a/linker/linker.cpp b/linker/linker.cpp
index 0db6e0f..b61e041 100755
--- a/linker/linker.cpp
+++ b/linker/linker.cpp
@@ -1031,10 +1031,6 @@
          * Section 4.7.1.10 "Dynamic relocations"
          * R_AARCH64_COPY may only appear in executable objects where e_type is
          * set to ET_EXEC.
-         *
-         * FLAG_EXE is set for both ET_DYN and ET_EXEC executables.
-         * We should explicitly disallow ET_DYN executables from having
-         * R_AARCH64_COPY relocations.
          */
         DL_ERR("%s R_AARCH64_COPY relocations are not supported", si->name);
         return -1;
@@ -1216,14 +1212,13 @@
             break;
         case R_ARM_COPY:
             /*
+             * ET_EXEC is not supported so this should not happen.
+             *
              * http://infocenter.arm.com/help/topic/com.arm.doc.ihi0044d/IHI0044D_aaelf.pdf
              *
              * Section 4.7.1.10 "Dynamic relocations"
              * R_ARM_COPY may only appear in executable objects where e_type is
              * set to ET_EXEC.
-             *
-             * We explicitly disallow ET_DYN executables from having
-             * R_ARM_COPY relocations.
              */
             DL_ERR("%s R_ARM_COPY relocations are not supported", si->name);
             return -1;