SuccessChanges

Summary

  1. Revert "PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI." (details)
  2. AMDGPU/GlobalISel: Fix assert on copy to vcc (details)
  3. AMDGPU/GlobalISel: Make s16 phi legal (details)
  4. [flang] Add options to control IMPLICIT NONE (details)
Commit 1de43bd6df74fb4cd8c25020afa1dd14ecf08527 by Raphael Isemann
Revert "PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI."

This reverts commit 87c5437afd273e909e0fed3389de7531d5452ea5.

The commit includes several headers in the middle of a function, which
breaks pretty much everything.
The file was modifiedllvm/include/llvm/DebugInfo/PDB/PDBExtras.h (diff)
The file was modifiedllvm/lib/DebugInfo/PDB/PDBExtras.cpp (diff)
Commit 53162567090251e98b14f979ee2f69669cec3ddd by Matthew.Arsenault
AMDGPU/GlobalISel: Fix assert on copy to vcc

This was trying to constrain a physical register. By the verifier's
understanding, it's impossible to have a 1-bit copy to vcc/vcc_lo so
don't try to handle physregs.
The file was modifiedllvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-copy.mir (diff)
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp (diff)
Commit 37894ba6612bf24060f38001122946108a770bc8 by Matthew.Arsenault
AMDGPU/GlobalISel: Make s16 phi legal

If we were to have an operation with an s16 def that needs to be
executed in a waterfall loop, not having s16 legal would place an
avoidable burden on RegBankSelect to widen it.
The file was modifiedllvm/test/CodeGen/AMDGPU/GlobalISel/legalize-phi.mir (diff)
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp (diff)
Commit 08c7d570d30bb9568fb2219db7b1b8a9532559dd by tkeith
[flang] Add options to control IMPLICIT NONE

Add `-fimplicit-none-type-always` to treat each specification-part
like it has `IMPLICIT NONE`. This is helpful for enforcing good Fortran
programming practices. We might consider something similar for
`IMPLICIT NONE(EXTERNAL)` as well.

Add `-fimplicit-none-type-never` to ignore occurrences of `IMPLICIT NONE`
and `IMPLICIT NONE(TYPE)`. This is to handle cases like the one below,
which violates the standard but it accepted by some compilers:
```
subroutine s(a, n)
  implicit none
  real :: a(n)
  integer :: n
end
```

Differential Revision: https://reviews.llvm.org/D85363
The file was addedflang/test/Semantics/implicit10.f90
The file was modifiedflang/lib/Semantics/resolve-names.cpp (diff)
The file was modifiedflang/tools/f18/f18.cpp (diff)
The file was addedflang/test/Semantics/implicit09.f90
The file was modifiedflang/documentation/Extensions.md (diff)
The file was modifiedflang/include/flang/Common/Fortran-features.h (diff)