[llvm] r187819 - Change private functions of LTOCodeGenerator from ret-false-on-succ to ret-true-on-succ.

Eric Christopher echristo at gmail.com
Tue Aug 6 15:08:46 PDT 2013


Should probably add something into the coding standard if we're going
to start moving existing code over to one or the other instead of just
changing it, I didn't see anything authorative on the thread.

-eric

On Tue, Aug 6, 2013 at 3:05 PM, Shuxin Yang <shuxin.llvm at gmail.com> wrote:
> This was discussed the other day.  Most people don't like
> return-false-on-succ.
> My code are all return-true-on-succ, when integrate with the
> LTOCodeGenerator,
> it will be extremely confusing.
>
>
> On 8/6/13 3:00 PM, Eric Christopher wrote:
>>
>> ... why?
>>
>> -eric
>>
>> On Tue, Aug 6, 2013 at 2:51 PM, Shuxin Yang <shuxin.llvm at gmail.com> wrote:
>>>
>>> Author: shuxin_yang
>>> Date: Tue Aug  6 16:51:21 2013
>>> New Revision: 187819
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=187819&view=rev
>>> Log:
>>> Change private functions of LTOCodeGenerator from ret-false-on-succ to
>>> ret-true-on-succ.
>>>
>>> Modified:
>>>      llvm/trunk/tools/lto/LTOCodeGenerator.cpp
>>>
>>> Modified: llvm/trunk/tools/lto/LTOCodeGenerator.cpp
>>> URL:
>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/lto/LTOCodeGenerator.cpp?rev=187819&r1=187818&r2=187819&view=diff
>>>
>>> ==============================================================================
>>> --- llvm/trunk/tools/lto/LTOCodeGenerator.cpp (original)
>>> +++ llvm/trunk/tools/lto/LTOCodeGenerator.cpp Tue Aug  6 16:51:21 2013
>>> @@ -159,7 +159,7 @@ bool LTOCodeGenerator::setCodePICModel(l
>>>
>>>   bool LTOCodeGenerator::writeMergedModules(const char *path,
>>>                                             std::string &errMsg) {
>>> -  if (determineTarget(errMsg))
>>> +  if (!determineTarget(errMsg))
>>>       return true;
>>>
>>>     // Run the verifier on the merged modules.
>>> @@ -213,7 +213,7 @@ bool LTOCodeGenerator::compile_to_file(c
>>>     }
>>>
>>>     objFile.keep();
>>> -  if (genResult) {
>>> +  if (!genResult) {
>>>       sys::fs::remove(Twine(Filename));
>>>       return true;
>>>     }
>>> @@ -252,7 +252,7 @@ const void* LTOCodeGenerator::compile(si
>>>
>>>   bool LTOCodeGenerator::determineTarget(std::string &errMsg) {
>>>     if (_target != NULL)
>>> -    return false;
>>> +    return true;
>>>
>>>     // if options were requested, set them
>>>     if (!_codegenOptions.empty())
>>> @@ -267,7 +267,7 @@ bool LTOCodeGenerator::determineTarget(s
>>>     // create target machine from info for merged modules
>>>     const Target *march = TargetRegistry::lookupTarget(TripleStr,
>>> errMsg);
>>>     if (march == NULL)
>>> -    return true;
>>> +    return false;
>>>
>>>     // The relocation model is actually a static member of TargetMachine
>>> and
>>>     // needs to be set before the TargetMachine is instantiated.
>>> @@ -300,7 +300,7 @@ bool LTOCodeGenerator::determineTarget(s
>>>     _target = march->createTargetMachine(TripleStr, _mCpu, FeatureStr,
>>> Options,
>>>                                          RelocModel, CodeModel::Default,
>>>                                          CodeGenOpt::Aggressive);
>>> -  return false;
>>> +  return true;
>>>   }
>>>
>>>   void LTOCodeGenerator::
>>> @@ -391,8 +391,8 @@ void LTOCodeGenerator::applyScopeRestric
>>>   /// Optimize merged modules using various IPO passes
>>>   bool LTOCodeGenerator::generateObjectFile(raw_ostream &out,
>>>                                             std::string &errMsg) {
>>> -  if (this->determineTarget(errMsg))
>>> -    return true;
>>> +  if (!this->determineTarget(errMsg))
>>> +    return false;
>>>
>>>     Module* mergedModule = _linker.getModule();
>>>
>>> @@ -435,7 +435,7 @@ bool LTOCodeGenerator::generateObjectFil
>>>     if (_target->addPassesToEmitFile(codeGenPasses, Out,
>>>                                      TargetMachine::CGFT_ObjectFile)) {
>>>       errMsg = "target file type not supported";
>>> -    return true;
>>> +    return false;
>>>     }
>>>
>>>     // Run our queue of passes all at once now, efficiently.
>>> @@ -444,7 +444,7 @@ bool LTOCodeGenerator::generateObjectFil
>>>     // Run the code generator, and write assembly file
>>>     codeGenPasses.run(*mergedModule);
>>>
>>> -  return false; // success
>>> +  return true;
>>>   }
>>>
>>>   /// setCodeGenDebugOptions - Set codegen debugging options to aid in
>>> debugging
>>>
>>>
>>> _______________________________________________
>>> 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