[PATCH] D128474: [BOLT] Support multiple parents for split jump table

Rafael Auler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 24 16:47:21 PDT 2022


rafauler added inline comments.


================
Comment at: bolt/lib/Core/BinaryContext.cpp:788
       if (opts::Verbosity > 2) {
         outs() << "BOLT-WARNING: Multiple fragments access same jump table: "
+               << JT->Parents[0]->getPrintName() << "; "
----------------
Warnings are printed to errs() and not outs()

BOLT-INFO is output to outs(). If this information is just reporting something on the input binary, consider converting it to INFO instead of WARNING, specially if this is supported.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128474/new/

https://reviews.llvm.org/D128474



More information about the llvm-commits mailing list