tree 6a462bc1d5486cbf6b04f7c1515502c73afcf372
parent cb78cfca14a0df486633253e3c2d9eb73cd37a4a
author Michael Kruse <llvm-project@meinersbur.de> 1613009278 -0600
committer Copybara-Service <copybara-worker@google.com> 1613736299 -0800

[Polly] Avoid "using namespace llvm" in public headers. NFC.

"using namespace" pollutes the namespace of every file that includes
such a header and universally considered a bad thing. Even the variant

    namespace polly {
      using namespace llvm;
    }

(previously used by LoopGenerators.h) imports more symbols than the file
is in control of. The header may include a fixed set of files from LLVM,
but the header itself may by be included together with other headers
from LLVM. For instance, LLVM's MemorySSA.h and Polly's ScopInfo.h both
declare a class 'MemoryAccess' which may conflict.

Instead of prefixing everything in Polly's header files, this patch adds
'using' statements to import only the symbols that are actually
referenced in Polly. This approach is also used by MLIR to import
commonly used symbols into the mlir namespace.

This patch also puts the symbols declared in IslNodeBuilder.h into the
Polly namespace to also be able to use the imported symbols.

GitOrigin-RevId: 91ca9adc9edfba164b579d02c5fe0a7a24cfdd4e