This is minimal change for frontend required to have "hello world" compiled
and working on x32 target (x86_64-linux-gnux32).
Note that this patch for backend is also required:
http://reviews.llvm.org/D4181
Details
Diff Detail
Event Timeline
lib/Basic/Targets.cpp | ||
---|---|---|
3262–3267 | I think chaining the assignment here similar to above might be nicer. | |
3270–3272 | The two data layouts are identical except for the pointer size. Can you factor that out so that its more obvious and less redundant please? | |
lib/Driver/ToolChains.cpp | ||
2977 | Can you add a check that arch is x86_64? Right now, you could make up ppc64---gnux32 if Im not mistaken. | |
lib/Driver/Tools.cpp | ||
6729 | Can you do this inside the x86_64 case? Or is there a reason to make this its own case? | |
6928 | Can you extend this to check the arch? | |
7041 | Here as well. |
More importantly, this should wait for the llvmdev discussion much like the
other x32 patchset.
llvmdev thread: http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-July/074451.html
I'll add test.
Added tests.
Redesigned findBiarchMultilibs for 3 arch (32,x32,64) support to pass more
tests.
Hi, it seems llvmdev discussion started week ago is over with some good comments made.
Can you please review this patch so we can proceed with these 2 patches merge and continue work on the next ones?
Please let me know if you think anything else is required to proceed here.
Thanks.
I think chaining the assignment here similar to above might be nicer.