[PATCH] D53204: [ELF] [HEXAGON] Let input determine ELF header e_flags

[Sid Manning via Phabricator via llvm-commits]

sidneym updated this revision to Diff 169496.
sidneym added a comment.

Update following Rui's suggestions.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D53204

Files:
  ELF/Arch/Hexagon.cpp


Index: ELF/Arch/Hexagon.cpp
===================================================================
--- ELF/Arch/Hexagon.cpp
+++ ELF/Arch/Hexagon.cpp
@@ -55,8 +55,19 @@
   NoneRel = R_HEX_NONE;
 }
 
-// Support V60 only at the moment.
-uint32_t Hexagon::calcEFlags() const { return 0x60; }
+uint32_t Hexagon::calcEFlags() const {
+  assert(!ObjectFiles.empty());
+
+  // The architecture revision must always be equal to or greater than
+  // greatest revision in the list of inputs.
+  uint32_t Ret = 0;
+  for (InputFile *F : ObjectFiles) {
+    uint32_t EFlags = cast<ObjFile<ELF32LE>>(F)->getObj().getHeader()->e_flags;
+    if (EFlags > Ret)
+      Ret = EFlags;
+  }
+  return Ret;
+}
 
 static uint32_t applyMask(uint32_t Mask, uint32_t Data) {
   uint32_t Result = 0;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53204.169496.patch
Type: text/x-patch
Size: 772 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181012/29c22a4c/attachment.bin>