dmgreen added a comment.

In D108138#2948995 <https://reviews.llvm.org/D108138#2948995>, @david-arm wrote:

> In D108138#2948975 <https://reviews.llvm.org/D108138#2948975>, @dmgreen wrote:
>
>>> I'm under the impression that the vectoriser has a policy of never making 
>>> scalar transformations
>>
>> I'm not sure what you mean. I've not looked into the details, but it could 
>> presumably be done as some sort of VPlan transformation, possibly in the 
>> constructions of vplans to treat switches like multiple icmp's/branches?
>
> Hi @dmgreen, I just meant that if LV makes a scalar transformation prior to 
> legality/cost-model checks, then for some reason we don't vectorise, we then 
> end up with a changed scalar body without any vectorisation.

Oh yeah, that makes sense. I was wondering if we could teach VPlan to treat 
them as ICmp/Br without having to actually transform the IR, just doing it as 
part of constructing the VPlan.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108138/new/

https://reviews.llvm.org/D108138

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to