SuccessChanges

Summary

  1. [NFC] Add missing include to fix modules build (details)
  2. Fix Wdocumentation warning. NFCI. (details)
  3. [PowerPC] Regenerate test for D52431 (details)
  4. Prune Pass.h include from DataLayout.h. NFCI (details)
  5. Prune include of DataLayout.h from include/clang/Basic/TargetInfo.h. NFC (details)
  6. [test] Merge Driver/as-w-warnings.c into as-no-warnings.c (details)
  7. SemaExceptionSpec - silence static analyzer getAs<> null dereference (details)
  8. [NFC][CVP] Add `shl` no-wrap deduction test coverage (details)
  9. [Clang] Add VerboseOutputStream to CompilerInstance (details)
  10. SemaTemplateDeduction - silence static analyzer getAs<> null dereference (details)
  11. Reverted r375425 as it broke some buildbots. (details)
  12. [X86] Rename matchBitOpReduction to matchScalarReduction. NFCI. (details)
  13. [AMDGPU] Select AGPR in PHI operand legalization (details)
  14. AMDGPU: Slightly restructure m0 init code (details)
  15. AMDGPU: Stop adding m0 implicit def to SGPR spills (details)
  16. AMDGPU: Erase redundant redefs of m0 in SIFoldOperands (details)
  17. AMDGPU: Use CopyToReg for interp intrinsic lowering (details)
Commit ffd87cb46a7becc61247bacdb9dc1718856fcdd9 by Raphael Isemann
[NFC] Add missing include to fix modules build
This header doesn't seem to be parsable on its own and breaks the module
build therefore with the following error:
While building module 'LLVM_Backend' imported from
llvm-project/llvm/lib/CodeGen/MachineScheduler.cpp:14: In file included
from <module-includes>:62:
llvm-project/llvm/include/llvm/CodeGen/MachinePipeliner.h:91:20: error:
declaration of 'AAResultsWrapperPass' must be imported from module
'LLVM_Analysis.AliasAnalysis' before it is required
   AU.addRequired<AAResultsWrapperPass>();
                  ^
llvm-project/llvm/include/llvm/Analysis/AliasAnalysis.h:1157:7: note:
previous declaration is here class AAResultsWrapperPass : public
FunctionPass {
     ^ llvm-project/llvm/lib/CodeGen/MachineScheduler.cpp:14:10: fatal
error: could not build module 'LLVM_Backend'
~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 errors generated.
llvm-svn: 375433
The file was modifiedllvm/include/llvm/CodeGen/MachinePipeliner.h
Commit 67854aa33dda6452b04f51715d558c0db705fbdb by llvm-dev
Fix Wdocumentation warning. NFCI.
llvm-svn: 375434
The file was modifiedclang/include/clang/AST/ExprCXX.h
Commit 0a803dd8229cc712ca594680ba47b631beee9a13 by llvm-dev
[PowerPC] Regenerate test for D52431
llvm-svn: 375435
The file was modifiedllvm/test/CodeGen/PowerPC/pr42492.ll
Commit 1f43ea41c330d1b0580b601e55f641f03ddced04 by bjorn.a.pettersson
Prune Pass.h include from DataLayout.h. NFCI
Summary: Reduce include dependencies by no longer including Pass.h from
DataLayout.h. That include seemed irrelevant to DataLayout, as well as
being irrelevant to several users of DataLayout.
Reviewers: rnk
Reviewed By: rnk
Subscribers: mehdi_amini, hiraditya, cfe-commits, llvm-commits
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D69261
llvm-svn: 375436
The file was modifiedllvm/include/llvm/IR/PassManager.h
The file was modifiedllvm/tools/llvm-profdata/llvm-profdata.cpp
The file was modifiedclang/lib/Tooling/AllTUsExecution.cpp
The file was modifiedllvm/unittests/IR/ModuleTest.cpp
The file was modifiedllvm/include/llvm/IR/Module.h
The file was modifiedllvm/lib/Transforms/Utils/CanonicalizeAliases.cpp
The file was modifiedllvm/include/llvm/IR/DataLayout.h
Commit 78424e5f8417f5014006e951dbe813d092cb567f by bjorn.a.pettersson
Prune include of DataLayout.h from include/clang/Basic/TargetInfo.h. NFC
Summary: Use a forward declaration of DataLayout instead of including
DataLayout.h in clangs TargetInfo.h. This reduces include dependencies
toward DataLayout.h (and other headers such as DerivedTypes.h, Type.h
that is included by DataLayout.h).
Needed to move implemantation of TargetInfo::resetDataLayout from
TargetInfo.h to TargetInfo.cpp.
Reviewers: rnk
Reviewed By: rnk
Subscribers: jvesely, nhaehnle, cfe-commits, llvm-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69262
llvm-svn: 375438
The file was modifiedclang/lib/Frontend/InitPreprocessor.cpp
The file was modifiedclang/lib/Basic/Targets/AMDGPU.cpp
The file was modifiedclang/lib/Basic/TargetInfo.cpp
The file was modifiedclang/unittests/AST/DeclTest.cpp
The file was modifiedclang/include/clang/Basic/TargetInfo.h
Commit 0cfd9e5b580f06d63be7054ba4efa0524011f096 by maskray
[test] Merge Driver/as-w-warnings.c into as-no-warnings.c
For -integrated-as RUN lines we can remove -target.
llvm-svn: 375439
The file was modifiedclang/test/Driver/as-no-warnings.c
The file was removedclang/test/Driver/as-w-option.c
Commit afb163f53ad567d72757e05c95e4059d3eb8de24 by llvm-dev
SemaExceptionSpec - silence static analyzer getAs<> null dereference
warnings. NFCI.
The static analyzer is warning about potential null dereferences, but in
these cases we should be able to use castAs<> directly and if not assert
will fire for us.
llvm-svn: 375440
The file was modifiedclang/lib/Sema/SemaExceptionSpec.cpp
Commit ca7f4d8b85b0e1f2f7ab4afab8a31b6aee5964cb by lebedev.ri
[NFC][CVP] Add `shl` no-wrap deduction test coverage
llvm-svn: 375441
The file was addedllvm/test/Transforms/CorrelatedValuePropagation/shl.ll
Commit 87cb734c04beab4731b51ff6763f5e63a9e604d6 by scott
[Clang] Add VerboseOutputStream to CompilerInstance
Remove one instance of a hardcoded output stream in
CompilerInstance::ExecuteAction. There are still other cases of output
being hard-coded to standard streams in ExecuteCompilerInvocation, but
this patch covers the case when no flags like -version or -help are
passed, namely the "X warnings and Y errors generated." diagnostic.
Differential Revision: https://reviews.llvm.org/D53768
llvm-svn: 375442
The file was modifiedclang/unittests/Frontend/OutputStreamTest.cpp
The file was modifiedclang/include/clang/Frontend/CompilerInstance.h
The file was modifiedclang/lib/Frontend/CompilerInstance.cpp
Commit 502a509e478a5e6625b58d87b484aacd2387c64d by llvm-dev
SemaTemplateDeduction - silence static analyzer getAs<> null dereference
warnings. NFCI.
The static analyzer is warning about potential null dereferences, but in
these cases we should be able to use castAs<> directly and if not assert
will fire for us.
llvm-svn: 375443
The file was modifiedclang/lib/Sema/SemaTemplateDeduction.cpp
Commit 8f2dac471ad9acb751f8424e3c2dd13d5a709e15 by sander.desmalen
Reverted r375425 as it broke some buildbots.
llvm-svn: 375444
The file was modifiedllvm/lib/CodeGen/RegUsageInfoCollector.cpp
The file was modifiedllvm/lib/Target/ARM/ARMFrameLowering.cpp
The file was modifiedllvm/lib/CodeGen/TargetFrameLoweringImpl.cpp
The file was modifiedllvm/lib/Target/AArch64/AArch64FrameLowering.cpp
The file was modifiedllvm/include/llvm/CodeGen/TargetFrameLowering.h
The file was modifiedllvm/lib/Target/AArch64/AArch64MachineFunctionInfo.h
The file was modifiedllvm/test/DebugInfo/MIR/X86/live-debug-values-reg-copy.mir
The file was modifiedllvm/lib/CodeGen/LiveDebugValues.cpp
The file was modifiedllvm/lib/Target/ARM/ARMFrameLowering.h
The file was removedllvm/test/CodeGen/AArch64/wrong-callee-save-size-after-livedebugvariables.mir
Commit 7c15c4fb1745eb80d034f1ce3e2313b4c900bd23 by llvm-dev
[X86] Rename matchBitOpReduction to matchScalarReduction. NFCI.
This doesn't need to be just for bitops, but the ops do need to be fully
associative.
llvm-svn: 375445
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
Commit 33092194f2cefecc75b0fd90ea21843e3550d206 by Stanislav.Mekhanoshin
[AMDGPU] Select AGPR in PHI operand legalization
If a PHI defines AGPR legalize its operands to AGPR. At the moment we
can get an AGPR PHI with VGPR operands. I am not aware of any problems
as it seems to be handled gracefully in RA, but this is not right
anyway.
It also slightly decreases VGPR pressure in some cases because we do not
have to a copy via VGPR.
Differential Revision: https://reviews.llvm.org/D69206
llvm-svn: 375446
The file was modifiedllvm/lib/Target/AMDGPU/SIInstrInfo.cpp
The file was modifiedllvm/test/CodeGen/AMDGPU/mfma-loop.ll
Commit b5234b64af83cb607e358eb77555f7f30ec0ced4 by Matthew.Arsenault
AMDGPU: Slightly restructure m0 init code
This will allow using another operation to produce the glue in a future
change.
llvm-svn: 375447
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp
Commit dd6cf159bab7cfa5dd61670d5048dab48c934eba by Matthew.Arsenault
AMDGPU: Stop adding m0 implicit def to SGPR spills
r375293 removed the SGPR spilling with scalar stores path, so this is no
longer necessary. This also always had the defect of adding the def even
when this path wasn't in use.
llvm-svn: 375448
The file was modifiedllvm/test/CodeGen/AMDGPU/indirect-addressing-term.ll
The file was modifiedllvm/lib/Target/AMDGPU/SIInstrInfo.cpp
The file was modifiedllvm/test/CodeGen/AMDGPU/stack-slot-color-sgpr-vgpr-spills.mir
Commit 8ebbf25cb1e9b2c95903917b2aa72363e5b20a42 by Matthew.Arsenault
AMDGPU: Erase redundant redefs of m0 in SIFoldOperands
Only handle simple inter-block redefs of m0 to the same value. This
avoids interference from redefs of m0 in SILoadStoreOptimzer. I was
initially teaching that pass to ignore redefs of m0, but having them not
exist beforehand is much simpler.
This is in preparation for deleting the current special m0 handling in
SIFixSGPRCopies to allow the register coalescer to handle the difficult
cases.
llvm-svn: 375449
The file was modifiedllvm/lib/Target/AMDGPU/SIFoldOperands.cpp
The file was addedllvm/test/CodeGen/AMDGPU/fold-operands-remove-m0-redef.mir
Commit 38038f116f7b948a700e0edc4d3687c7b7fda926 by Matthew.Arsenault
AMDGPU: Use CopyToReg for interp intrinsic lowering
This doesn't use the default value, so doesn't benefit from the hack to
help optimize it.
llvm-svn: 375450
The file was modifiedllvm/test/CodeGen/AMDGPU/llvm.amdgcn.interp.f16.ll
The file was modifiedllvm/lib/Target/AMDGPU/SIISelLowering.cpp