<div dir="ltr">OK, that sounds great. Here's the patch to revert the offending line:<div><a href="https://ghc.haskell.org/trac/ghc/ticket/7942#comment:54">https://ghc.haskell.org/trac/ghc/ticket/7942#comment:54</a><br></div><div><br></div><div>And I'll prepare another patch to implement switching on 3.6 shortly. </div><div><br></div><div>I'm definitely in support of the LLVM-with-GHC plan!</div><div>Cheers</div><div>Luke</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 20, 2014 at 7:26 PM, Austin Seipp <span dir="ltr"><<a href="mailto:austin@well-typed.com" target="_blank">austin@well-typed.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">(Resending to list)<br>
<span class="im HOEnZb"><br>
Yes, switching on the version would be preferable since we detect the<br>
tool version in a few places already if it's possible. Or at least we<br>
should back out the regressing change right now until we ship LLVM<br>
with GHC as I proposed.<br>
<br>
</span><div class="HOEnZb"><div class="h5">On Thu, Nov 20, 2014 at 3:49 PM, Luke Iannini <<a href="mailto:lukexipd@gmail.com">lukexipd@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Nov 20, 2014 at 1:43 PM, Luke Iannini <<a href="mailto:lukexipd@gmail.com">lukexipd@gmail.com</a>> wrote:<br>
>><br>
>> Whoops, that's my fault; it shouldn't have slipped into the patch — this<br>
>> is required for LLVM 3.6 (which I was using to work on my ARM64 patch), but<br>
>> clearly isn't backwards compatible.<br>
>><br>
>> I wonder what's best to do for this? Would it be correct to switch on LLVM<br>
>> versions or should this be considered a bug in LLVM? (it was the only change<br>
>> needed for 3.6)<br>
><br>
> (I mean to say, it was the only change needed atop Ben's work on 3.5 support<br>
> in <a href="https://phabricator.haskell.org/D155" target="_blank">https://phabricator.haskell.org/D155</a>)<br>
><br>
>><br>
>><br>
>> I will submit a ticket to revert that line in the meantime.<br>
>><br>
>> Thanks!<br>
>> Luke<br>
>><br>
>> On Thu, Nov 20, 2014 at 2:26 AM, Dr. ERDI Gergo <<a href="mailto:gergo@erdi.hu">gergo@erdi.hu</a>> wrote:<br>
>>><br>
>>> Hi,<br>
>>><br>
>>> Tests fail on master when run the optllvm way, e.g.:<br>
>>><br>
>>> /usr/bin/opt: /tmp/ghc16190_0/ghc16190_2.ll:611:25: error: expected<br>
>>> 'global' or 'constant'<br>
>>> @newCAF$alias = private alias i8* @newCAF<br>
>>><br>
>>> I think it's the change in d87fa34:<br>
>>><br>
>>> index 7307725..cdc407c 100644<br>
>>> --- a/compiler/llvmGen/Llvm/PpLlvm.hs<br>
>>> +++ b/compiler/llvmGen/Llvm/PpLlvm.hs<br>
>>> @@ -80,7 +80,7 @@ ppLlvmGlobal (LMGlobal var@(LMGlobalVar _ _ link x a c)<br>
>>> dat) =<br>
>>>          const_link = case c of<br>
>>>            Global   -> ppr link <+> text "global"<br>
>>>            Constant -> ppr link <+> text "constant"<br>
>>> -          Alias    -> text "alias" <+> ppr link<br>
>>> +          Alias    -> ppr link <+> text "alias"<br>
>>><br>
>>>      in ppAssignment var $ const_link <+> rhs <> sect <> align<br>
>>>         $+$ newLine<br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>><br>
>>>   .--= ULLA! =-----------------.<br>
>>>    \     <a href="http://gergo.erdi.hu" target="_blank">http://gergo.erdi.hu</a>   \<br>
>>>     `---= <a href="mailto:gergo@erdi.hu">gergo@erdi.hu</a> =-------'<br>
>>> If it ain't broke, I can fix it.<br>
>>> _______________________________________________<br>
>>> ghc-devs mailing list<br>
>>> <a href="mailto:ghc-devs@haskell.org">ghc-devs@haskell.org</a><br>
>>> <a href="http://www.haskell.org/mailman/listinfo/ghc-devs" target="_blank">http://www.haskell.org/mailman/listinfo/ghc-devs</a><br>
>><br>
>><br>
><br>
><br>
> _______________________________________________<br>
> ghc-devs mailing list<br>
> <a href="mailto:ghc-devs@haskell.org">ghc-devs@haskell.org</a><br>
> <a href="http://www.haskell.org/mailman/listinfo/ghc-devs" target="_blank">http://www.haskell.org/mailman/listinfo/ghc-devs</a><br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Regards,<br>
<br>
Austin Seipp, Haskell Consultant<br>
Well-Typed LLP, <a href="http://www.well-typed.com/" target="_blank">http://www.well-typed.com/</a><br>
</font></span></blockquote></div><br></div>