[lld/mac] Delete incorrect FIXME
authorNico Weber <thakis@chromium.org>
Wed, 23 Jun 2021 20:24:41 +0000 (16:24 -0400)
committerNico Weber <thakis@chromium.org>
Wed, 23 Jun 2021 20:25:34 +0000 (16:25 -0400)
"""Bitcode symbols only exist before LTO runs, and only serve the purpose of
resolving visibility so LTO can better optimize. Running LTO creates ObjFiles
from BitcodeFiles, and those ObjFiles contain regular Defined symbols (with
isec set and all) that will replace the bitcode symbols. So things should
(hopefully) work as-is :)"""

  -- https://reviews.llvm.org/rGdbbc8d8333f29cf4ad6f4793da1adf71bbfdac69#inline-6081

lld/MachO/UnwindInfoSection.cpp

index e1e8d30..f82395b 100644 (file)
@@ -297,8 +297,6 @@ static void addEntriesForFunctionsWithoutUnwindInfo(
   // Add explicit "has no unwind info" entries for all global and local symbols
   // without unwind info.
   auto markNoUnwindInfo = [&cuVector, &hasUnwindInfo](const Defined *d) {
-    // FIXME: The d->isec null check might be incorrect for symbols
-    // from LTO files.
     if (d->isLive() && d->isec && isCodeSection(d->isec)) {
       Ptr ptr = d->getVA();
       if (!hasUnwindInfo.count(ptr))