[llvm] [LV] Vectorize histogram operations (PR #99851)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 30 02:57:21 PDT 2024
================
@@ -30,6 +31,7 @@
#include "llvm/Transforms/Utils/BasicBlockUtils.h"
#include "llvm/Transforms/Utils/LoopUtils.h"
#include "llvm/Transforms/Utils/ScalarEvolutionExpander.h"
+#include "llvm/Transforms/Vectorize/LoopVectorizationLegality.h"
----------------
fhahn wrote:
It looks like VPHistogramRecipe only needs `Update` of `HistogramInfo` and that only for retrieving the legacy cost. Ideally we would include `VPHistogramRecipe::computeCost` in this patch, avoiding the need for accessing `Update`.
If that isn't easily possibly, can we just store `Update` (with a comment what it is used for)?
https://github.com/llvm/llvm-project/pull/99851
More information about the llvm-commits
mailing list