[llvm] r270865 - Use new interface in Triple /NFC

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu May 26 10:50:44 PDT 2016


Nope - no idea, I haven't looked at it closely at all - just figured since
it broke a buildbot (I assume you diagnosed that correctly - but do realize
that many buildbots are flakey/complicated... so it may not be your fault
if you're pretty sure this is No Functional Change)

On Thu, May 26, 2016 at 10:49 AM, Xinliang David Li <davidxl at google.com>
wrote:

> I am a little puzzled why this change can make any change in compiler
> behavior. Any idea?
>
> David
>
> On Thu, May 26, 2016 at 10:38 AM, David Blaikie <dblaikie at gmail.com>
> wrote:
>
>> (I see this was reverted - but when it goes back it it could use a test
>> case)
>>
>> On Thu, May 26, 2016 at 9:28 AM, Xinliang David Li via llvm-commits <
>> llvm-commits at lists.llvm.org> wrote:
>>
>>> Author: davidxl
>>> Date: Thu May 26 11:28:01 2016
>>> New Revision: 270865
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=270865&view=rev
>>> Log:
>>> Use new interface in Triple /NFC
>>>
>>> Modified:
>>>     llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
>>>
>>> Modified:
>>> llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
>>> URL:
>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp?rev=270865&r1=270864&r2=270865&view=diff
>>>
>>> ==============================================================================
>>> --- llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
>>> (original)
>>> +++ llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp Thu
>>> May 26 11:28:01 2016
>>> @@ -790,7 +790,7 @@ static void createIRLevelProfileFlagVari
>>>        INSTR_PROF_QUOTE(IR_LEVEL_PROF_VERSION_VAR));
>>>    IRLevelVersionVariable->setVisibility(GlobalValue::DefaultVisibility);
>>>    Triple TT(M.getTargetTriple());
>>> -  if (TT.isOSBinFormatMachO())
>>> +  if (!TT.supportsCOMDAT())
>>>      IRLevelVersionVariable->setLinkage(GlobalValue::WeakAnyLinkage);
>>>    else
>>>      IRLevelVersionVariable->setComdat(M.getOrInsertComdat(
>>>
>>>
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160526/b08cb8be/attachment.html>


More information about the llvm-commits mailing list