search for: combinelevel

...don't seem to work with constant arguments, i.e. changing the call in add-with-overflow.ll to: %t = call {i32, i1} @llvm.sadd.with.overflow.i32(i32 0, i32 0) causes the following exception when running the codegen tests: llc: DAGCombiner.cpp:646: void<unnamed>::DAGCombiner::Run(llvm::CombineLevel): Assertion `N->getValueType(0) == RV.getValueType() && N->getNumValues() == 1 && "Type mismatch"' failed. 0 llc 0x00000000010601ef 1 llc 0x00000000010604ec 2 libc.so.6 0x00007f58a754df60 3 libc.so.6 0x00007f58a754ded5 gsignal + 53 4 libc.so...

...ex: DAGCombiner.cpp =================================================================== --- DAGCombiner.cpp (revision 269301) +++ DAGCombiner.cpp (working copy) @@ -1251,6 +1251,10 @@ //===----------------------------------------------------------------------===// void DAGCombiner::Run(CombineLevel AtLevel) { + + if (OptLevel == CodeGenOpt::None) + return; + // set the instance variables, so that the various visit routines may use it. Level = AtLevel; LegalOperations = Level >= AfterLegalizeVectorOps; It would most likely break some CodeGen tests since it would have an impact...

To extwnd on Martin 's explanation : In factor(), levels are the unique input values and labels the unique output values. So the function levels() actually displays the labels. Cheers Joris On 15 Jun 2017 17:15, "Martin Maechler" <maechler at stat.math.ethz.ch> wrote: >>>>> Paul Johnson <pauljohn32 at gmail.com> >>>>> on Wed, 14 Jun

Applied. Thanks, Zoltan! -bw On Tue, Dec 9, 2008 at 1:12 PM, Zoltan Varga <vargaz at gmail.com> wrote: > Hi, > > Attached is the final version of the patch, adding the requested > FIXME. If this is ok, can > somebody check it in ? > > thanks > > Zoltan > > On Tue, Dec 9, 2008 at 9:58 PM,

...uot;male", "Female")) > levels(xf) <- c("Male", "Male", "Male", "Female") > I think it is quirky that `levels<-.factor` allows the duplicated > labels, whereas factor does not. > I wrote a function rockchalk::combineLevels to simplify combining > levels, but most of the students here like plyr::mapvalues to do it. > The use of levels() can be tricky because one must enumerate all > values, not just the ones being changed. > But I do understand Martin's point. Its been this way 25 yea...

...le")) >>> levels(xf) <- c("Male", "Male", "Male", "Female") > >>> I think it is quirky that `levels<-.factor` allows the duplicated >>> labels, whereas factor does not. > >>> I wrote a function rockchalk::combineLevels to simplify combining >>> levels, but most of the students here like plyr::mapvalues to do it. >>> The use of levels() can be tricky because one must enumerate all >>> values, not just the ones being changed. > >>> But I do understand Martin's point. Its...

Hi, Attached is the final version of the patch, adding the requested FIXME. If this is ok, can somebody check it in ? thanks Zoltan On Tue, Dec 9, 2008 at 9:58 PM, Bill Wendling <isanbard at gmail.com> wrote: > On Tue, Dec 9, 2008 at 6:11 AM, Zoltan Varga <vargaz at gmail.com> wrote: >> Hi, >>

...quot;Male", "Man", "male", "Female")) levels(xf) <- c("Male", "Male", "Male", "Female") I think it is quirky that `levels<-.factor` allows the duplicated labels, whereas factor does not. I wrote a function rockchalk::combineLevels to simplify combining levels, but most of the students here like plyr::mapvalues to do it. The use of levels() can be tricky because one must enumerate all values, not just the ones being changed. But I do understand Martin's point. Its been this way 25 years, it won't change. :). > C...

..."Female")) >> levels(xf) <- c("Male", "Male", "Male", "Female") >> I think it is quirky that `levels<-.factor` allows the duplicated >> labels, whereas factor does not. >> I wrote a function rockchalk::combineLevels to simplify combining >> levels, but most of the students here like plyr::mapvalues to do it. >> The use of levels() can be tricky because one must enumerate all >> values, not just the ones being changed. >> But I do understand Martin's point. Its been...

...;> levels(xf) <- c("Male", "Male", "Male", "Female") >> >>>> I think it is quirky that `levels<-.factor` allows the duplicated >>>> labels, whereas factor does not. >> >>>> I wrote a function rockchalk::combineLevels to simplify combining >>>> levels, but most of the students here like plyr::mapvalues to do it. >>>> The use of levels() can be tricky because one must enumerate all >>>> values, not just the ones being changed. >> >>>> But I do understand Mart...

...======================================================== > --- DAGCombiner.cpp (revision 269301) > +++ DAGCombiner.cpp (working copy) > @@ -1251,6 +1251,10 @@ > //===----------------------------------------------------------------------===// > > void DAGCombiner::Run(CombineLevel AtLevel) { > + > + if (OptLevel == CodeGenOpt::None) > + return; > + > // set the instance variables, so that the various visit routines may use it. > Level = AtLevel; > LegalOperations = Level >= AfterLegalizeVectorOps; > > It would most likely break so...