[PATCH] D83177: [llvm-reduce] Reducing call operand bundles
Nick Desaulniers via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 6 12:31:56 PDT 2020
nickdesaulniers added inline comments.
================
Comment at: llvm/tools/llvm-reduce/deltas/ReduceOperandBundles.cpp:34
+
+using namespace llvm;
+
----------------
lebedev.ri wrote:
> nickdesaulniers wrote:
> > if you move this up, then you don't need to wrap the forward declaration of `class Module` above.
> Hold up, that's actually bogus.
> https://godbolt.org/z/0qTwwz
or just include the appropriate header for `llvm::Module` and forget forward declaration?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83177/new/
https://reviews.llvm.org/D83177
More information about the llvm-commits
mailing list