FailedChanges

Summary

  1. [asan] Return true from instrumentModule createSanitizerCtorAndInitFunctions always change the module.
  2. DebugInfo: Fix msan use-of-uninitialized exposed by r374600
  3. [llvm-profdata] Make "malformed-ptr-to-counter-array.test" textual As pointed out in https://reviews.llvm.org/D66979 post-commit, making this test textual would make it more maintainable. Differential Revision: https://reviews.llvm.org/D68718
  4. [X86] Fold a VTRUNCS/VTRUNCUS+store into a saturating truncating store. We already did this for VTRUNCUS with a specific combination of types. This extends this to VTRUNCS and handles any types where a truncating store is legal.
  5. [X86] Add test case showing missing opportunity to fold vmovsdb into a store after type legalization. NFC
  6. DebugInfo: Reduce the scope of some variables related to debug_ranges emission Minor tidy up/NFC
  7. [libFuzzer] Don't prefix absolute paths in fuchsia. The ExecuteCommand function in fuchsia used to prefix the getOutputFile for each command run with the artifact_prefix flag if it was available, because fuchsia components don't have a writable working directory. However, if a file with a global path is provided, fuchsia should honor that. An example of this is using the global /tmp directory to store stuff. In fuchsia it ended up being translated to data///tmp, whereas we want to make sure it is using /tmp (which is available to components using the isolated-temp feature). To test this I made the change, compiled fuchsia with this toolchain and ran a fuzzer with the -fork=1 flag (that mode makes use of the /tmp directory). I also tested that normal fuzzing workflow was not affected by this. Author: charco (Marco Vanotti) Differential Revision: https://reviews.llvm.org/D68774
  8. gn build: (manually) merge r374606 better
  9. gn build: Merge r235758
  10. gn build: Cmanually) merge r374590
  11. [AMDGPU] Use GCN prefix in dpp_combine.mir. NFC.
  12. [AMDGPU] link dpp pseudos and real instructions on gfx10 This defaults to zero fi operand, but we do not expose it anyway. Should we expose it later it needs to be added to the pseudo. This enables dpp combining on gfx10. Differential Revision: https://reviews.llvm.org/D68888
  13. [lit] Small cleanups in main.py * Extract separate function for running tests from main * Push single-usage imports to point of usage * Remove unnecessary sys.exit(0) calls Reviewed By: rnk Differential Revision: https://reviews.llvm.org/D68836
  14. [lit] Change regex filter to ignore case Make regex filter `--filter=REGEX` option more lenient via `re.IGNORECASE`. Reviewed By: yln Differential Revision: https://reviews.llvm.org/D68834
  15. DebugInfo: Use base address selection entries for debug_loc Unify the range and loc emission (for both DWARFv4 and DWARFv5 style lists) and take advantage of that unification to use strategic base addresses for loclists. Differential Revision: https://reviews.llvm.org/D68620
Revision 374623 by Vitaly Buka:
[asan] Return true from instrumentModule

createSanitizerCtorAndInitFunctions always change the module.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cppllvm.src/lib/Transforms/Instrumentation/AddressSanitizer.cpp
Revision 374619 by dblaikie:
DebugInfo: Fix msan use-of-uninitialized exposed by r374600
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/DebugInfo/DWARF/DWARFDebugLoc.cppllvm.src/lib/DebugInfo/DWARF/DWARFDebugLoc.cpp
Revision 374617 by Vedant Kumar:
[llvm-profdata] Make "malformed-ptr-to-counter-array.test" textual

As pointed out in https://reviews.llvm.org/D66979 post-commit, making
this test textual would make it more maintainable.

Differential Revision: https://reviews.llvm.org/D68718
Change TypePath in RepositoryPath in Workspace
The file was removed/llvm/trunk/test/tools/llvm-profdata/Inputs/malformed-ptr-to-counter-array.profrawllvm.src/test/tools/llvm-profdata/Inputs/malformed-ptr-to-counter-array.profraw
The file was modified/llvm/trunk/test/tools/llvm-profdata/malformed-ptr-to-counter-array.testllvm.src/test/tools/llvm-profdata/malformed-ptr-to-counter-array.test
Revision 374615 by ctopper:
[X86] Fold a VTRUNCS/VTRUNCUS+store into a saturating truncating store.

We already did this for VTRUNCUS with a specific combination of
types. This extends this to VTRUNCS and handles any types where
a truncating store is legal.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelLowering.cppllvm.src/lib/Target/X86/X86ISelLowering.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-trunc.llllvm.src/test/CodeGen/X86/avx512-trunc.ll
Revision 374614 by ctopper:
[X86] Add test case showing missing opportunity to fold vmovsdb into a store after type legalization. NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-trunc.llllvm.src/test/CodeGen/X86/avx512-trunc.ll
Revision 374613 by dblaikie:
DebugInfo: Reduce the scope of some variables related to debug_ranges emission

Minor tidy up/NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cppllvm.src/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp
Revision 374612 by jakehehrlich:
[libFuzzer] Don't prefix absolute paths in fuchsia.

The ExecuteCommand function in fuchsia used to prefix the
getOutputFile for each command run with the artifact_prefix flag if
it was available, because fuchsia components don't have a writable working
directory. However, if a file with a global path is provided, fuchsia
should honor that.

An example of this is using the global /tmp directory to store stuff.
In fuchsia it ended up being translated to data///tmp, whereas we want
to make sure it is using /tmp (which is available to components using the
isolated-temp feature).

To test this I made the change, compiled fuchsia with this toolchain and
ran a fuzzer with the -fork=1 flag (that mode makes use of the /tmp
directory). I also tested that normal fuzzing workflow was not affected
by this.

Author: charco (Marco Vanotti)
Differential Revision: https://reviews.llvm.org/D68774
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerUtilFuchsia.cppcompiler-rt.src/lib/fuzzer/FuzzerUtilFuchsia.cpp
Revision 374611 by nico:
gn build: (manually) merge r374606 better
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/gn/secondary/libunwind/src/BUILD.gnllvm.src/utils/gn/secondary/libunwind/src/BUILD.gn
Revision 374610 by gnsyncbot:
gn build: Merge r235758
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/gn/secondary/libunwind/src/BUILD.gnllvm.src/utils/gn/secondary/libunwind/src/BUILD.gn
Revision 374608 by nico:
gn build: Cmanually) merge r374590
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/gn/secondary/llvm/lib/Target/targets.gnillvm.src/utils/gn/secondary/llvm/lib/Target/targets.gni
The file was added/llvm/trunk/utils/gn/secondary/llvm/tools/llvm-exegesis/lib/Mipsllvm.src/utils/gn/secondary/llvm/tools/llvm-exegesis/lib/Mips
The file was added/llvm/trunk/utils/gn/secondary/llvm/tools/llvm-exegesis/lib/Mips/BUILD.gnllvm.src/utils/gn/secondary/llvm/tools/llvm-exegesis/lib/Mips/BUILD.gn
The file was modified/llvm/trunk/utils/gn/secondary/llvm/unittests/BUILD.gnllvm.src/utils/gn/secondary/llvm/unittests/BUILD.gn
The file was added/llvm/trunk/utils/gn/secondary/llvm/unittests/tools/llvm-exegesis/Mipsllvm.src/utils/gn/secondary/llvm/unittests/tools/llvm-exegesis/Mips
The file was added/llvm/trunk/utils/gn/secondary/llvm/unittests/tools/llvm-exegesis/Mips/BUILD.gnllvm.src/utils/gn/secondary/llvm/unittests/tools/llvm-exegesis/Mips/BUILD.gn
Revision 374607 by rampitec:
[AMDGPU] Use GCN prefix in dpp_combine.mir. NFC.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/dpp_combine.mirllvm.src/test/CodeGen/AMDGPU/dpp_combine.mir
Revision 374604 by rampitec:
[AMDGPU] link dpp pseudos and real instructions on gfx10

This defaults to zero fi operand, but we do not expose it
anyway. Should we expose it later it needs to be added to
the pseudo.

This enables dpp combining on gfx10.

Differential Revision: https://reviews.llvm.org/D68888
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPUMCInstLower.cppllvm.src/lib/Target/AMDGPU/AMDGPUMCInstLower.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/VOP1Instructions.tdllvm.src/lib/Target/AMDGPU/VOP1Instructions.td
The file was modified/llvm/trunk/lib/Target/AMDGPU/VOP2Instructions.tdllvm.src/lib/Target/AMDGPU/VOP2Instructions.td
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/atomic_optimizations_local_pointer.llllvm.src/test/CodeGen/AMDGPU/atomic_optimizations_local_pointer.ll
The file was added/llvm/trunk/test/CodeGen/AMDGPU/dpp_combine.llllvm.src/test/CodeGen/AMDGPU/dpp_combine.ll
Revision 374602 by yln:
[lit] Small cleanups in main.py

* Extract separate function for running tests from main
* Push single-usage imports to point of usage
* Remove unnecessary sys.exit(0) calls

Reviewed By: rnk

Differential Revision: https://reviews.llvm.org/D68836
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/lit/lit/main.pyllvm.src/utils/lit/lit/main.py
Revision 374601 by yln:
[lit] Change regex filter to ignore case

Make regex filter `--filter=REGEX` option more lenient via
`re.IGNORECASE`.

Reviewed By: yln

Differential Revision: https://reviews.llvm.org/D68834
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/lit/lit/cl_arguments.pyllvm.src/utils/lit/lit/cl_arguments.py
The file was modified/llvm/trunk/utils/lit/lit/main.pyllvm.src/utils/lit/lit/main.py
The file was modified/llvm/trunk/utils/lit/tests/selecting.pyllvm.src/utils/lit/tests/selecting.py
Revision 374600 by dblaikie:
DebugInfo: Use base address selection entries for debug_loc

Unify the range and loc emission (for both DWARFv4 and DWARFv5 style lists) and take advantage of that unification to use strategic base addresses for loclists.

Differential Revision: https://reviews.llvm.org/D68620
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cppllvm.src/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/debug-loclists.llllvm.src/test/CodeGen/X86/debug-loclists.ll
The file was modified/llvm/trunk/test/DebugInfo/X86/sret.llllvm.src/test/DebugInfo/X86/sret.ll