SuccessChanges

Summary

  1. clang-tidy doc: modernize-make-unique has an autofix (details)
  2. [InstCombine] Add tests for PR44242 (details)
  3. [InstCombine] Don't rewrite phi-of-bitcast when the phi has other users (details)
  4. Fix external-names.c test when separator is \\ (details)
Commit e8c9110b56b516a22b41e95e347bc141814ab87c by sylvestre
clang-tidy doc: modernize-make-unique has an autofix
The file was modifiedclang-tools-extra/docs/clang-tidy/checks/list.rst
Commit d04e64a25a58f63af862df9fa305723560d5fded by nikita.ppv
[InstCombine] Add tests for PR44242
Differential Revision: https://reviews.llvm.org/D71260
The file was addedllvm/test/Transforms/InstCombine/pr44242.ll
Commit fb114694e939c0204ac356fc0e830332175cd008 by nikita.ppv
[InstCombine] Don't rewrite phi-of-bitcast when the phi has other users
Judging by the existing comments, this was the intention, but the
transform never actually checked if the existing phi's would be removed.
See https://bugs.llvm.org/show_bug.cgi?id=44242 for an example where
this causes much worse code generation on AMDGPU.
Differential Revision: https://reviews.llvm.org/D71209
The file was modifiedllvm/test/Transforms/InstCombine/pr44242.ll
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineCasts.cpp
Commit 0c7ca82161b519856bb1d54e181b578dc067cd3e by michael.platings
Fix external-names.c test when separator is \\
This fixes the following failure:
C:\[...]\llvm\tools\clang\test\VFS\external-names.c:34:26: error:
CHECK-DEBUG-EXTERNAL: expected string not found in input
// CHECK-DEBUG-EXTERNAL: ![[Num]] = !DIFile(filename:
"{{[^"]*}}Inputs{{.}}external-names.h"
                        ^
[...]
<stdin>:42:54: note: possible intended match here
!10 = !DIFile(filename:
"C:/[...]\\llvm\\tools\\clang\\test\\VFS\\Inputs\\external-names.h",
directory: "")
Differential Revision: https://reviews.llvm.org/D71991
The file was modifiedclang/test/VFS/external-names.c