SuccessChanges

Summary

  1. [clang-tidy] Fix altera-struct-pack-align crash for struct fields with incomplete type (details)
  2. [APInt][NFC] Fix typo vlalue->value (details)
  3. [RISCV][NFC] Correct alignment in scatter/gather tests (details)
  4. [AMDGPU] Tweak VOP3_INTERP16 profile (details)
  5. [clang] In DependencyCollector on Windows, ignore case and separators when discarding duplicate dependency file paths. (details)
  6. [X86] Don't dereference a dyn_cast<> - use a cast<> instead. NFCI. (details)
  7. [x86] update fma test with deprecated intrinsics; NFC (details)
Commit ab92a4c26f54170bf72706ad29c0fb151a177590 by jubnzv
[clang-tidy] Fix altera-struct-pack-align crash for struct fields with incomplete type

We can only use ASTContext::getTypeInfo for complete types.

This fixes bugzilla issue 50313.

Reviewed By: aaron.ballman

Differential Revision: https://reviews.llvm.org/D102569
The file was modifiedclang-tools-extra/clang-tidy/altera/StructPackAlignCheck.cpp
The file was addedclang-tools-extra/test/clang-tidy/checkers/altera-struct-pack-align-no-crash.cpp
Commit b6e4bfd18571b65bf3c537f52225d8ee6c2953c4 by weratt
[APInt][NFC] Fix typo vlalue->value

Reviewed By: fhahn

Differential Revision: https://reviews.llvm.org/D102618
The file was modifiedllvm/include/llvm/ADT/APInt.h
Commit cd73ce4b5e622bb4d71b41f9dab157200501d352 by fraser
[RISCV][NFC] Correct alignment in scatter/gather tests

This lays the groundwork for changes to alignment in D102493 to be more
apparent.
The file was modifiedllvm/test/Analysis/CostModel/RISCV/fixed-vector-scatter.ll
The file was modifiedllvm/test/Analysis/CostModel/RISCV/fixed-vector-gather.ll
Commit 472f856714fb1687b0727b3013f5d071d8fa86ae by jay.foad
[AMDGPU] Tweak VOP3_INTERP16 profile

Set the output register class based on the output type, instead of
hard-coding VGPR_32. I think this is more correct. It doesn't make any
difference at the moment because we use the same class for 16- and
32-bit results, but it might in future if we make more use of true
16-bit register classes.

Differential Revision: https://reviews.llvm.org/D102622
The file was modifiedllvm/lib/Target/AMDGPU/VOP3Instructions.td
Commit 6052a8a53559d667321637f7159353ab724a1141 by sylvain.audi
[clang] In DependencyCollector on Windows, ignore case and separators when discarding duplicate dependency file paths.

This patch removes duplicates also encountered in the output of clang-scan-deps when one same header file is encountered with different casing and/or different separators ('/' vs '\').

The case of separators can appear when the same file is included externally by
`#include <folder/file.h>`

whereas a file from the same folder does
`#include "file.h"`

Under Windows, clang computes the paths using '/' from the include directive, the `\` from the -I options, and the concatenations use the native `\`, leading to internal paths containing a mix of both separators.

Differential Revision: https://reviews.llvm.org/D102339
The file was modifiedclang/lib/Frontend/DependencyFile.cpp
The file was addedclang/test/Frontend/dependency-gen-windows-duplicates.c
Commit 41587466aaf239d061ad084114ec749cecbb2966 by llvm-dev
[X86] Don't dereference a dyn_cast<> - use a cast<> instead. NFCI.

dyn_cast<> can return null if the cast fails, by using cast<> we assert that the cast is correct helping to avoid a potential null dereference.
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
Commit 8854b27b198c1c499ca174369fcc649060afb075 by spatel
[x86] update fma test with deprecated intrinsics; NFC

All of the CHECK lines should be identical to before,
but without any of the x86-specific calls that were
replaced with generic FMA long ago.

The file still has value because it shows a miscompile
as demonstrated in D90901, but we probably need to
add tests with FMF to make that explicit without
losing coverage.
The file was modifiedllvm/test/CodeGen/X86/avx2-fma-fneg-combine.ll