[PATCH] D42949: [X86] Promote 16-bit cmovs to 32-bit.

Roman Lebedev via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 6 07:00:08 PST 2018


Ah, ok, makes sense, the repo was specified, so https://reviews.llvm.org/H270
 auto-subscribed the list, since the differential's subject did not
contain [private].
I'm not seeing anything about diff's visibility in Herald Rule Conditions,
so not sure it can be avoided..

On Tue, Feb 6, 2018 at 5:46 PM, Craig Topper <craig.topper at gmail.com> wrote:
> Arcanist seems to have subscribed it without me telling it to.
>
> On Tue, Feb 6, 2018 at 4:15 AM Roman Lebedev <lebedev.ri at gmail.com> wrote:
>>
>> Is it expected that this differential is seemingly view-restricted?
>> (if yes, might have been good to avoid the mailing list post too)
>>
>> On Tue, Feb 6, 2018 at 11:09 AM, Craig Topper via Phabricator via
>> llvm-commits <llvm-commits at lists.llvm.org> wrote:
>> > craig.topper created this revision.
>> > Herald added a subscriber: llvm-commits.
>> >
>> > Repository:
>> >   rL LLVM
>> >
>> > https://reviews.llvm.org/D42949
>> >
>> > Files:
>> >   lib/CodeGen/SelectionDAG/DAGCombiner.cpp
>> >   lib/Target/X86/X86ISelLowering.cpp
>> >   test/CodeGen/X86/avx512-mask-op.ll
>> >   test/CodeGen/X86/avx512-schedule.ll
>> >   test/CodeGen/X86/cmov-promotion.ll
>> >
>> >
>> > _______________________________________________
>> > llvm-commits mailing list
>> > llvm-commits at lists.llvm.org
>> > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>> >
>
> --
> ~Craig


More information about the llvm-commits mailing list