search for: l374

Hi, AFAICT there are two places where zext instructions may get folded into PHI nodes. One is FoldPHIArgZextsIntoPHI and the other is the more generic FoldPHIArgOpIntoPHI. Now, the former only handles PHIs with more than 2 incoming values, while the latter only handles casts where the source type is legal. This means that for an PHI node with two incoming i8 values, both resulting from `zext i1

...so we get the benefit > of further execution tests, while still sending fail-mail, etc for the > warning?) > > That sounds reasonable, although the buildbot implementation seems to be pretty focused on make: https://github.com/buildbot/buildbot/blob/master/master/buildbot/steps/shell.py#L374 I'm guessing that between CMake and Ninja, buildbot will probably need some minor changes. > On Tue, Mar 22, 2016 at 5:00 PM, David Jones via cfe-dev < > cfe-dev at lists.llvm.org> wrote: > >> Greetings, >> >> I would like to propose adding a buildbot which bu...

...IR, so we might as well allow transforming to that type? > > I'm not sure yet, but there's a chance that change might induce problems > (infinite loops) with this: > https://github.com/llvm-mirror/llvm/blob/master/lib/Transfor > ms/InstCombine/InstCombineSimplifyDemanded.cpp#L374 > > > On Sun, Jan 29, 2017 at 3:09 PM, Björn Steinbrink via llvm-dev < > llvm-dev at lists.llvm.org> wrote: > >> Hi, >> >> AFAICT there are two places where zext instructions may get folded into >> PHI nodes. One is FoldPHIArgZextsIntoPHI and the other i...

Could we change the buildbot config to treat warnings-as-errors-that-continue in the build step? (so we get the benefit of further execution tests, while still sending fail-mail, etc for the warning?) On Tue, Mar 22, 2016 at 5:00 PM, David Jones via cfe-dev < cfe-dev at lists.llvm.org> wrote: > Greetings, > > I would like to propose adding a buildbot which builds with -Werror. The

...well allow transforming to that type? >> >> I'm not sure yet, but there's a chance that change might induce problems >> (infinite loops) with this: >> https://github.com/llvm-mirror/llvm/blob/master/lib/Transfor >> ms/InstCombine/InstCombineSimplifyDemanded.cpp#L374 >> >> >> On Sun, Jan 29, 2017 at 3:09 PM, Björn Steinbrink via llvm-dev < >> llvm-dev at lists.llvm.org> wrote: >> >>> Hi, >>> >>> AFAICT there are two places where zext instructions may get folded into >>> PHI nodes. One is Fo...

Greetings, I would like to propose adding a buildbot which builds with -Werror. The reason for a new buildbot in this configuration is twofold: 1. It helps users who track and release from ToT, because they (generally) seem to build with -Werror. Speaking from experience :-), new warnings tend to crop up in a large range of commits, and end up blocking one or more of these downstream users.