[ELF] Remove FIXME(s) that are already fixed.
authorShankar Easwaran <shankare@codeaurora.org>
Sat, 21 Feb 2015 15:51:54 +0000 (15:51 +0000)
committerShankar Easwaran <shankare@codeaurora.org>
Sat, 21 Feb 2015 15:51:54 +0000 (15:51 +0000)
FIXME code was left around in few places where its already been taken care of.

This removes the FIXME's that are not needed.

llvm-svn: 230139

lld/lib/ReaderWriter/ELF/Atoms.h

index b95c782..be1daaa 100644 (file)
@@ -124,9 +124,8 @@ public:
 
   StringRef name() const override { return _name; }
 
-  // FIXME: What distinguishes a symbol in ELF that can help decide if the
-  // symbol is undefined only during build and not runtime? This will make us
-  // choose canBeNullAtBuildtime and canBeNullAtRuntime.
+  // A symbol in ELF can be undefined at build time if the symbol is a undefined
+  // weak symbol.
   CanBeNull canBeNull() const override {
     if (_symbol->getBinding() == llvm::ELF::STB_WEAK)
       return CanBeNull::canBeNullAtBuildtime;
@@ -192,7 +191,6 @@ public:
   // FIXME: Need to revisit this in future.
   Interposable interposable() const override { return interposeNo; }
 
-  // FIXME: What ways can we determine this in ELF?
   Merge merge() const override {
     if (_symbol->getBinding() == llvm::ELF::STB_WEAK)
       return mergeAsWeak;