[all-commits] [llvm/llvm-project] 2b223b: [mlir] Fix warnings when compiling with Clang 9.0

[Alexandre Ganea via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 2b223bd1c7d5bdd019aac4f540e8b60d339348a3
      https://github.com/llvm/llvm-project/commit/2b223bd1c7d5bdd019aac4f540e8b60d339348a3
  Author: Alexandre Ganea <alexandre.ganea at ubisoft.com>
  Date:   2020-01-01 (Wed, 01 Jan 2020)

  Changed paths:
    M mlir/lib/IR/AsmPrinter.cpp

  Log Message:
  -----------
  [mlir] Fix warnings when compiling with Clang 9.0

Fixes: warning: comparison of integers of different signs: 'const unsigned int' and '(anonymous namespace)::OperationPrinter::(anonymous enum at F:\llvm-project\mlir\lib\IR\AsmPrinter.cpp:1444:3)' [-Wsign-compare]


  Commit: 316f6003ef2b3361919bbc421b26357bf10129ee
      https://github.com/llvm/llvm-project/commit/316f6003ef2b3361919bbc421b26357bf10129ee
  Author: Alexandre Ganea <alexandre.ganea at ubisoft.com>
  Date:   2020-01-01 (Wed, 01 Jan 2020)

  Changed paths:
    M mlir/CMakeLists.txt

  Log Message:
  -----------
  [mlir] Fix linking with LLD

The issue is that /WHOLEARCHIVE is interpreted differently in LLD, which needs the same exact path as the .lib; whereas link.exe can take the library name, withoutout a path or extension, if that was already supplied on the cmd-line. I'll write a follow-up patch to fix the issue in LLD.


  Commit: 6656e961c08393c3949412ef945ade0272b66fca
      https://github.com/llvm/llvm-project/commit/6656e961c08393c3949412ef945ade0272b66fca
  Author: Alexandre Ganea <alexandre.ganea at ubisoft.com>
  Date:   2020-01-01 (Wed, 01 Jan 2020)

  Changed paths:
    M mlir/lib/Dialect/Linalg/Analysis/DependenceAnalysis.cpp
    M mlir/tools/mlir-tblgen/RewriterGen.cpp

  Log Message:
  -----------
  [mlir] Fix compilation warnings

Fixes:
- (MSVC) F:\llvm-project\mlir\lib\Dialect\Linalg\Analysis\DependenceAnalysis.cpp(103): warning C4551: function call missing argument list
- (Clang) tools\mlir\lib\Dialect\SPIRV\SPIRVCanonicalization.inc(232,1): warning: unused function 'populateWithGenerated' [-Wunused-function]


  Commit: 92b68c1937cd065a2fc44d18c1099de7da19b356
      https://github.com/llvm/llvm-project/commit/92b68c1937cd065a2fc44d18c1099de7da19b356
  Author: Alexandre Ganea <alexandre.ganea at ubisoft.com>
  Date:   2020-01-01 (Wed, 01 Jan 2020)

  Changed paths:
    M llvm/lib/Support/StringRef.cpp
    M llvm/unittests/ADT/StringRefTest.cpp
    M polly/lib/Analysis/ScopDetection.cpp

  Log Message:
  -----------
  [polly][Support] Un-break polly tests
Previously, the polly unit tests were stuck in a infinite loop.
There was an edge case in StringRef::count() introduced by 9f6b13e5cce96066d7262d224c971d93c2724795, where an empty 'Str' would cause the function to never exit.
Also fixed usage in polly.


Compare: https://github.com/llvm/llvm-project/compare/fb170fd652bf...92b68c1937cd