[PATCH] D123678: [polly] migrate -polly-show to the new pass manager

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 16 13:15:49 PDT 2022


Meinersbur added inline comments.


================
Comment at: polly/include/polly/ScopGraphPrinter.h:28
+using namespace polly;
+using namespace llvm;
+
----------------
RKSimon wrote:
> @YangKeao Can you remove the 'using namespace' from the header and move them internally to each source file? This might be the cause of the namespace resolution build failures on:
> 
> https://github.com/compiler-explorer/compiler-workflows/runs/6452927622
I was just looking into it when I noticed you already changed it. Thanks you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123678



More information about the llvm-commits mailing list