[PATCH] D129473: [LV] Make scalable-vectorization not work if there is a specified scalable pragma hint.

Mon Jul 11 04:00:57 PDT 2022

fakepaper56 added a comment.

In D129473#3641972 <https://reviews.llvm.org/D129473#3641972>, @sdesmalen wrote:

> After this change, the behaviour no longer matches the behaviour described above on lines 118-120:
>
>   // If the metadata doesn't explicitly specify whether to enable scalable
>   // vectorization, then decide based on the following criteria (increasing
>   // level of priority):
>   //  - Target default
>   //  - Metadata width
>   //  - Force option (always overrides)
>
> because it no longer overrides the loop hint if one is specified.

Sorry, I don't understand your idea. I think the change not only reserves the priority of the three cases but also realize the predicate "If the metadata doesn't explicitly specify whether to enable scalable".

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129473