[llvm-commits] [patch] Tautology?

Chris Lattner clattner at apple.com
Thu Mar 18 14:12:17 PDT 2010


I tracked this down and fixed the underlying issue.

-Chris

On Mar 17, 2010, at 11:20 PM, Chris Lattner wrote:

> 
> On Mar 17, 2010, at 11:16 PM, Evan Cheng wrote:
> 
>> 
>> On Mar 17, 2010, at 11:00 PM, Chris Lattner wrote:
>> 
>>> Yeah, that is weird.  Evan, you added this in r40033, do you happen to know what you meant here?
>> 
>> No. I am blanking. Looks like copy-n-paste bug. I think all the != should be ==?
> 
> No idea!  Please make whatever fix you think is right, optionally just removing the code as dead.
> 
> -Chris
> 
>> 
>> Evan
>> 
>>> 
>>> -Chris
>>> 
>>> On Mar 17, 2010, at 7:49 PM, Zhongxing Xu wrote:
>>> 
>>>> Index: utils/TableGen/Record.cpp
>>>> ===================================================================
>>>> --- utils/TableGen/Record.cpp    (版本 98703)
>>>> +++ utils/TableGen/Record.cpp    (工作副本)
>>>> @@ -644,20 +644,6 @@
>>>>   DagInit *LHSs = dynamic_cast<DagInit*>(LHS);
>>>>   DagInit *RHSs = dynamic_cast<DagInit*>(RHS);
>>>>   if (LHSs && RHSs) {
>>>> -      DefInit *LOp = dynamic_cast<DefInit*>(LHSs->getOperator());
>>>> -      DefInit *ROp = dynamic_cast<DefInit*>(RHSs->getOperator());
>>>> -      if (LOp->getDef() != ROp->getDef()) {
>>>> -        bool LIsOps =
>>>> -          LOp->getDef()->getName() == "outs" ||
>>>> -          LOp->getDef()->getName() != "ins" ||
>>>> -          LOp->getDef()->getName() != "defs";
>>>> -        bool RIsOps =
>>>> -          ROp->getDef()->getName() == "outs" ||
>>>> -          ROp->getDef()->getName() != "ins" ||
>>>> -          ROp->getDef()->getName() != "defs";
>>>> -        if (!LIsOps || !RIsOps)
>>>> -          throw "Concated Dag operators do not match!";
>>>> -      }
>>>>     std::vector<Init*> Args;
>>>>     std::vector<std::string> ArgNames;
>>>>     for (unsigned i = 0, e = LHSs->getNumArgs(); i != e; ++i) {
>>>> 
>>>> 
>>>> LOp->getDef()->getName() != "ins" || LOp->getDef()->getName() != "defs";
>>>> will always be true.
>>>> _______________________________________________
>>>> llvm-commits mailing list
>>>> llvm-commits at cs.uiuc.edu
>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>> 
>> 
> 





More information about the llvm-commits mailing list