r374288 - Recommit "[Clang] Pragma vectorize_width() implies vectorize(enable)"

Michael Kruse via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 25 15:18:48 PDT 2019


@sjoerdmeijer

Before recommitting, please re-open the patch review.

Michael

Am Do., 24. Okt. 2019 um 18:45 Uhr schrieb Jordan Rupprecht
<rupprecht at google.com>:
>
> Reverted in 6d424a161bf3e52730371da0b9439ed93a8ce406 due to the issue described here. Should hopefully be a trivial fix forward.
>
> On Tue, Oct 22, 2019 at 2:46 PM Michael Kruse <llvm-commits at meinersbur.de> wrote:
>>
>> Am Mo., 21. Okt. 2019 um 23:44 Uhr schrieb Jordan Rupprecht
>> <rupprecht at google.com>:
>> > At any rate, it sounds like this is not a codegen bug at all, but just an over-eager warning?
>>
>> That interpretation is different from mine. Codgen emits the following
>> from vectorize(disable)
>>
>> !4 = !{!"llvm.loop.vectorize.enable", i1 true}
>>
>> which is is not what I'd expect.
>>
>> Michael


More information about the cfe-commits mailing list