[r3579] Clang fails to build ZDoom

Forum rules
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.

Post a reply

Smilies
:D :) :( :o :shock: :? 8-) :lol: :x :P :oops: :cry: :evil: :twisted: :roll: :wink: :geek: :ugeek: :!: :?: :idea: :arrow: :| :mrgreen: :3: :wub: >:( :blergh:
View more smilies

BBCode is OFF
Smilies are ON

Topic review
   

Expand view Topic review: [r3579] Clang fails to build ZDoom

Re: [r3579] Clang fails to build ZDoom

by MP2E » Tue Apr 24, 2012 5:12 am

Seems I somehow deleted a random line and it was included as part of my patch, creating an error with dumb. Gez found it and fixed it in r1365 of GZDoom, sorry about that.

(Also thanks for attributing the post to me)

EDIT: Also GZDoom builds perfectly now as well with clang

Re: [r3579] Clang fails to build ZDoom

by wildweasel » Sun Apr 22, 2012 10:39 pm

MP2E wrote:(By the way, I wrote the post/patch above I just hadn't logged in to the ZDoom forums for a while so I forgot my pass :X)
That little "issue" just got corrected, by the way - your post is now properly attributed to you.

Re: [r3579] Clang fails to build ZDoom

by MP2E » Sun Apr 22, 2012 10:20 pm

Yep! Works with stable release 3.0 and the 3.1 current svn branch. Not only does it work but it runs well :) Haven't bench-marked it against the GCC build in my linux distribution yet but that's planned.

(By the way, I wrote the post/patch above I just hadn't logged in to the ZDoom forums for a while so I forgot my pass :X)

Re: [r3579] Clang fails to build ZDoom

by randi » Sun Apr 22, 2012 8:47 pm

So clang has progressed enough to be able to compile ZDoom now? That's pretty impressive.

[r3579] Clang fails to build ZDoom

by MP2E » Sun Apr 22, 2012 7:44 pm

Clang fails to build ZDoom due to dumb using a variable named "restrict" which is a reserved keyword, and failing on some inline ASM that GCC uses. This was fixed using the patch below and doesn't effect anything other than compilation with clang.

http://pastebin.com/pvCFxw0U

Top