en Copyright 2015 Java http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#5a667c0e [llvm] Use nullptr instead of 0 (NFC)Identified with modernize-use-nullptr. List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Tue, 28 Dec 2021 16:52:25 +0000 Kazu Hirata <kazu@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#0b5051ce [llvm-reduce] Don't reuse SmallVector across calls to getAllMetadata()The SmallVector is not cleared in calls to getAllMetadata().Reviewed By: MeinersburDifferential Revision: https://reviews.llvm.org/D113808 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Fri, 12 Nov 2021 23:10:59 +0000 Arthur Eubanks <aeubanks@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#6f288bd7 [llvm-reduce] Count chunks by running a preliminary reductionHaving a separate counting method runs the risk of a mismatch betweenthe actual reduction method and the counting method.Instead, create an Oracle that always returns true for shouldKeep(), runthe reduction, and count how many times shouldKeep() was called. Themodule should not be modified if shouldKeep() always returns true.Reviewed By: MeinersburDifferential Revision: https://reviews.llvm.org/D113537 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Fri, 12 Nov 2021 02:45:59 +0000 Arthur Eubanks <aeubanks@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#77bc3ba3 [NFC][llvm-reduce] Cleanup typesUse Module& wherever possible.Since every reduction immediately turns Chunks into an Oracle, directly pass Oracle instead.Reviewed By: hansDifferential Revision: https://reviews.llvm.org/D111122 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Tue, 05 Oct 2021 07:05:37 +0000 Arthur Eubanks <aeubanks@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#d9562a8e [llvm-reduce] Reduce metadata references.The ReduceMetadata pass before this patch removed metadata on a per-MDNode (or NamedMDNode) basis. Either all references to an MDNode are kept, or all of them are removed. However, MDNodes are uniqued, meaning that references to MDNodes with the same data become references to the same MDNodes. As a consequence, e.g. tbaa references to the same type will all have the same MDNode reference and hence make it impossible to reduce only keeping metadata on those memory access for which they are interesting.Moreover, MDNodes can also be referenced by some intrinsics or other MDNodes. These references were not considered for removal leading to the possibility that MDNodes are not actually removed even if selected to be removed by the oracle.This patch changes ReduceMetadata to reduces based on removable metadata references instead. MDNodes without references implicitly dropped anyway. References by intrinsic calls should be removed by ReduceOperands or ReduceInstructions. References in other MDNodes cannot be removed as it would violate the immutability of MDNodes.Additionally, ReduceMetadata pass before this patch used `setMetadata(I, NULL)` to remove references, where `I` is the index in the array returned by `getAllMetadata`. However, `setMetadata` expects a MDKind (such as `MD_tbaa`) as first argument. `getAllMetadata` does not return those in consecutive order (otherwise it would not need to be a `std::pair` with `first` representing the MDKind).Reviewed By: aeubanks, swamulismDifferential Revision: https://reviews.llvm.org/D110534 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Wed, 29 Sep 2021 15:51:13 +0000 Michael Kruse <llvm-project@meinersbur.de> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#a39c7ab9 [NFCI][llvm-reduce] Cleanup Delta passes to use Oracle abstractionSummary:I think, this results in much more understandable/readable flow.At least the original logic was perhaps the most hard thing for me to grasp when taking an initial look on the delta passes.Reviewers: nickdesaulniers, dblaikie, diegotf, george.burgess.ivReviewed By: nickdesaulniersSubscribers: llvm-commitsTags: #llvmDifferential Revision: https://reviews.llvm.org/D83287 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Wed, 08 Jul 2020 09:05:15 +0000 Roman Lebedev <lebedev.ri@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#c4da7eec llvm-reduce: Fix inconsistencies between int/unsigned usage (standardize on int)llvm-svn: 372270 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Wed, 18 Sep 2019 22:30:25 +0000 David Blaikie <dblaikie@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#e0d9a0bd Fix -Wdocumentation warning - void function doesn't need a @returns. NFCI.llvm-svn: 371606 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Wed, 11 Sep 2019 11:12:00 +0000 Simon Pilgrim <llvm-dev@redking.me.uk> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp#477c1df4 llvm-reduce: Add pass to reduce MetadataPatch by Diego Treviño!Differential Revision: https://reviews.llvm.org/D65026llvm-svn: 371562 List of files: /llvm-project-15.0.7/llvm/tools/llvm-reduce/deltas/ReduceMetadata.cpp Tue, 10 Sep 2019 22:09:58 +0000 David Blaikie <dblaikie@gmail.com>