================
@@ -80,6 +80,21 @@ enum class RuleTy {
   ForVectorization,
 };
 
+/// Store the information about if corresponding direction vector was negated
----------------
sjoerdmeijer wrote:

I think duplicated direction vectors are always allowed. They don't add new or 
different information, so it shouldn't effect the interpretation of the 
dependence analysis in any way. The only thing that it affects is processing 
the same information again and again, so the only benefit of making them unique 
is to avoid that. But if keeping all entries makes the logic easier, there is a 
good reason to not make them unique. I think adding all the state here 
complicates things, and if a simple map of original to negated helps, you've 
certainly got my vote to simplify this. 

https://github.com/llvm/llvm-project/pull/133672
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to