[PATCH] D110383: [JITLink][NFC] Add TableManager to replace PerGraph...Builder pass

[luxufan via Phabricator via llvm-commits]

StephenFan added inline comments.


================
Comment at: llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h:1686
 
+static void visitEdge(LinkGraph &G, Block *B, Edge &E) {}
+
----------------
lhames wrote:
> StephenFan wrote:
> > I use gcc to compile llvm-project. And this function will cause the `defined but not used[-Wunused-function]` warning. Is there ways to suppress this warning ?
> That sounds like a broken warning. I would ignore it.
I tried to land this patch, but broke builds on some bots. See  e.g. https://lab.llvm.org/buildbot#builders/77/builds/10218


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110383