Fails to build on 10.10.5, maybe march=native issue? about homebrew-hhvm

[jwatzman]

Needs full logs -- can you run brew gist-logs hhvm please?

[Hywan]

I have an error 403 (Forbidden) from brew gist-logs hhvm :-/.

[wjywbs]

It's probably because some Mac CPUs do not have sse crc support? Using -march=native might not work: facebook/hhvm@03a6cc6. I used -msse4.2 instead to always be able to build.

[jwatzman]

@wjywbs possibly, is clang somehow picking the wrong arch for native?

@Hywan I need some sort of logs to look into this. The full output of brew install --verbose hhvm would also be useful.

[Hywan]

@jwatzman It's running :-).

[Hywan]

@jwatzman http://pastebin.com/raw.php?i=nsMh20kt

[bkarge]

@jwatzman, @wjywbs: thanks!! -msse4.2 in HPHPCompiler.cmake did the trick. No idea yet how to tell clang on MacOS to enable full native CPU support (this i5-5257U should support SSE4.2). Will try playing with -target some time...

[wjywbs]

Clang 3.6 doesn't recognize new broadwell CPUs. Clang 3.7 should support it.

llvm-mirror/llvm@da740f1
llvm-mirror/llvm@9f7761b

[jwatzman]

@wjywbs oh good find, thanks. I still think -msse4.2 is better than -march=native, since it both specifies exactly what the problem is, as well as doesn't necessarily make the binaries non-portable (which will make me more comfortable providing bottles built on my newish Mac Pro).

[jwatzman]

Does that make sense to you?

[wjywbs]

Yes, I also prefer -msse4.2 and I always used it before -march=native was added. Clang on linux has this same issue.

[jwatzman]

OK, diff out to change this, should be on github later today. You'll have to build with --HEAD to get it since it's a fix to HHVM itself, not this formula, but I may update the formula to patch it in later. (I'd like to provide bottles and this will help.)