AbortedChanges

Summary

  1. Revert "[ms] [llvm-ml] Add support for INCLUDE environment variable" (details)
  2. [clang-tidy] performance-unnecessary-copy-initialization: Remove the complete statement when the copied variable is unused. (details)
  3. [compiler-rt][hwasan] Decouple use of the TLS global for getting the shadow base and using the frame record feature (details)
  4. [llvm-tapi-diff] Apply stable sorting to output (details)
  5. [OpenMP][NFC] Precommit change to hide_mem_transfer_latency test flags (details)
  6. [Attributor] Set floating point loads and stores as nofree in AANoFreeFloating (details)
  7. [Polly][Matmul] Re-pack A in every iteration. (details)
  8. [clang-tidy] performance-unnecessary-copy-initialization: Look at the canonical type when checking for aliases. (details)
Commit 68d0db0b6d56ec900044539b30c458d896ea55ba by epastor
Revert "[ms] [llvm-ml] Add support for INCLUDE environment variable"

This reverts commit c43f413b01b021a8f7b6fce013296114fa92a245 due to Windows environment build breaks
The file was removedllvm/test/tools/llvm-ml/include_by_env_var_errors.asm
The file was modifiedllvm/tools/llvm-ml/Opts.td
The file was modifiedllvm/tools/llvm-ml/llvm-ml.cpp
The file was removedllvm/test/tools/llvm-ml/include_by_env_var.asm
Commit 5dbe3bf4b8dbb7e67d41c7c1360f15d512dd72a0 by flx
[clang-tidy] performance-unnecessary-copy-initialization: Remove the complete statement when the copied variable is unused.

It is not useful to keep the statement around and can lead to compiler
warnings when -Wall (-Wunused-variable specifically) turned on.

Differential Revision: https://reviews.llvm.org/D102175

Reviewed-by: ymandel
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/performance-unnecessary-copy-initialization.cpp
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/performance-unnecessary-copy-initialization-allowed-types.cpp
The file was modifiedclang-tools-extra/clang-tidy/performance/UnnecessaryCopyInitialization.cpp
The file was modifiedclang-tools-extra/clang-tidy/performance/UnnecessaryCopyInitialization.h
Commit 314c049142ed22154fbe37de3a566ef38b884fab by leonardchan
[compiler-rt][hwasan] Decouple use of the TLS global for getting the shadow base and using the frame record feature

This allows for using the frame record feature (which uses __hwasan_tls)
independently from however the user wants to access the shadow base, which
prior was only usable if shadow wasn't accessed through the TLS variable or ifuncs.

Frame recording can be explicitly set according to ShadowMapping::WithFrameRecord
in ShadowMapping::init. Currently, it is only enabled on Fuchsia and if TLS is
used, so this should mimic the old behavior.

Added an extra case to prologue.ll that covers this new case.

Differential Revision: https://reviews.llvm.org/D103841
The file was modifiedllvm/test/Instrumentation/HWAddressSanitizer/prologue.ll
The file was modifiedllvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
Commit 90a26a41e9ce16a4d471d25c2f7b36b5538fb4ce by Cyndy Ishida
[llvm-tapi-diff] Apply stable sorting to output

* For the output, the attributes within the target slice should be
  grouped by the input order, then sorted by value ordering.
This is to fix current ubuntu buildbot inconsistences.
The file was modifiedllvm/tools/llvm-tapi-diff/DiffEngine.cpp
Commit c70c30d6da0a0d46c45ca6b77d760216ddd420a7 by huberjn
[OpenMP][NFC] Precommit change to hide_mem_transfer_latency test flags
The file was modifiedllvm/test/Transforms/OpenMP/hide_mem_transfer_latency.ll
Commit 4c9471581fa07331b9ec8e4ab1ea312fba43c0b6 by huberjn
[Attributor] Set floating point loads and stores as nofree in AANoFreeFloating

Summary:
The current implementation of AANoFreeFloating will incorrectly list floating
point loads and stores as may-free. This prevents other attributor instances
like HeapToStack from pushing some allocations to the stack.

Reviewed By: jdoerfert

Differential Revision: https://reviews.llvm.org/D103975
The file was modifiedllvm/test/Transforms/Attributor/heap_to_stack.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/alignment.ll
The file was modifiedllvm/test/Transforms/Attributor/nofree.ll
The file was modifiedllvm/test/Transforms/Attributor/readattrs.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/profile.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/fp80.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/tail.ll
The file was modifiedllvm/test/Transforms/OpenMP/parallel_deletion.ll
The file was modifiedllvm/test/Transforms/Attributor/nonnull.ll
The file was modifiedllvm/test/Transforms/Attributor/norecurse.ll
The file was modifiedllvm/lib/Transforms/IPO/AttributorAttributes.cpp
The file was modifiedllvm/test/Transforms/Attributor/range.ll
The file was modifiedllvm/test/Transforms/Attributor/value-simplify.ll
The file was modifiedllvm/test/Transforms/Attributor/willreturn.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/dbg.ll
The file was modifiedllvm/test/Transforms/Attributor/nocapture-1.ll
The file was modifiedllvm/test/Transforms/OpenMP/parallel_region_merging.ll
The file was modifiedllvm/test/Transforms/Attributor/callbacks.ll
The file was modifiedllvm/test/Transforms/Attributor/memory_locations.ll
The file was modifiedllvm/test/Transforms/Attributor/IPConstantProp/openmp_parallel_for.ll
Commit a56bd7dec8da4348d847d53c96d8a30f4a821d36 by llvm-project
[Polly][Matmul] Re-pack A in every iteration.

Packed_A must be copied repeatedly, not just for the first iteration of
the outer tile.

This fixes llvm.org/PR50557
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_5.ll
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_9.ll
The file was modifiedpolly/include/polly/Support/ISLTools.h
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_7.ll
The file was modifiedpolly/lib/Transform/MatmulOptimizer.cpp
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_4.ll
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_12.ll
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_3.ll
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_8.ll
The file was modifiedpolly/test/ScheduleOptimizer/mat_mul_pattern_data_layout.ll
The file was modifiedpolly/lib/Support/ISLTools.cpp
The file was modifiedpolly/test/ScheduleOptimizer/mat_mul_pattern_data_layout_2.ll
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_6.ll
The file was modifiedpolly/test/ScheduleOptimizer/pattern-matching-based-opts_13.ll
Commit efa4dbc32ca9b7f3319edbcc6ac502ea962c8f0a by flx
[clang-tidy] performance-unnecessary-copy-initialization: Look at the canonical type when checking for aliases.

This fixes a false positive case where for instance a pointer is obtained and declared using `auto`.

Differential Revision: https://reviews.llvm.org/D103018

Reviewed-by: ymandel
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/performance-unnecessary-copy-initialization.cpp
The file was modifiedclang-tools-extra/clang-tidy/performance/UnnecessaryCopyInitialization.cpp