[all-commits] [llvm/llvm-project] 550d14: [SandboxIR] Fix warning when building on Windows w...

[Alexandre Ganea via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 550d148561002a9984ca57a5ba0ed39367052a52
      https://github.com/llvm/llvm-project/commit/550d148561002a9984ca57a5ba0ed39367052a52
  Author: Alexandre Ganea <alex_toresh at yahoo.fr>
  Date:   2025-04-08 (Tue, 08 Apr 2025)

  Changed paths:
    M llvm/lib/SandboxIR/Constant.cpp

  Log Message:
  -----------
  [SandboxIR] Fix warning when building on Windows with clang-cl. NFC.

This fixes several of these:
```
[1151/3381] Building CXX object lib\SandboxIR\CMakeFiles\LLVMSandboxIR.dir\Constant.cpp.obj
C:\git\llvm-project\llvm\lib\SandboxIR\Constant.cpp(331,52): warning: duplicate explicit instantiation of 'operator()' ignored as a Microsoft extension [-Wmicrosoft-template]
  331 |                   llvm::GlobalObject>::LLVMGVToGV::operator()(llvm::GlobalIFunc
      |                                                    ^
C:\git\llvm-project\llvm\include\llvm/SandboxIR/Constant.h(816,52): note: previous explicit instantiation is here
  816 |                   llvm::GlobalObject>::LLVMGVToGV::operator()(llvm::GlobalIFunc
      |                                                    ^
```


  Commit: 446e793d77286ea477c90778f28c168883526e56
      https://github.com/llvm/llvm-project/commit/446e793d77286ea477c90778f28c168883526e56
  Author: Alexandre Ganea <alex_toresh at yahoo.fr>
  Date:   2025-04-08 (Tue, 08 Apr 2025)

  Changed paths:
    M llvm/include/llvm/SandboxIR/Type.h

  Log Message:
  -----------
  [SandboxIR] Fix warning when building on Windows with clang-cl. NFC.

This fixes:
```
[1230/3381] Building CXX object lib\Transforms\Vectorize\CMakeFiles\LLVMVectorize.dir\SandboxVectorizer\VecUtils.cpp.obj
In file included from C:\git\llvm-project\llvm\lib\Transforms\Vectorize\SandboxVectorizer\VecUtils.cpp:9:
In file included from C:\git\llvm-project\llvm\include\llvm/Transforms/Vectorize/SandboxVectorizer/VecUtils.h:17:
C:\git\llvm-project\llvm\include\llvm/SandboxIR/Type.h(55,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
   55 |   friend class CallBase;           // For LLVMTy.
      |                ^
      |                ::llvm::
C:\git\llvm-project\llvm\include\llvm/SandboxIR/Type.h(60,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
   60 |   friend class CmpInst;            // For LLVMTy. TODO: Cleanup after
      |                ^
      |                ::llvm::
2 warnings generated.
```


  Commit: 661f90ad088fe9fad004ae6690bab061fb9a5f1d
      https://github.com/llvm/llvm-project/commit/661f90ad088fe9fad004ae6690bab061fb9a5f1d
  Author: Alexandre Ganea <alex_toresh at yahoo.fr>
  Date:   2025-04-08 (Tue, 08 Apr 2025)

  Changed paths:
    M llvm/include/llvm/SandboxIR/Context.h

  Log Message:
  -----------
  [SandboxIR] Fix warning when building on Windows with clang-cl. NFC.

This fixes:
```
[2295/3381] Building CXX object lib\Passes\CMakeFiles\LLVMPasses.dir\PassBuilder.cpp.obj
In file included from C:\git\llvm-project\llvm\lib\Passes\PassBuilder.cpp:368:
In file included from C:\git\llvm-project\llvm\include\llvm/Transforms/Vectorize/SandboxVectorizer/SandboxVectorizer.h:16:
C:\git\llvm-project\llvm\include\llvm/SandboxIR/Context.h(73,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
   73 |   friend class Region;            // For LLVMCtx.
      |                ^
      |                ::llvm::
1 warning generated.
```


Compare: https://github.com/llvm/llvm-project/compare/c4c0ff686563...661f90ad088f

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications