[PATCH] D34562: [MSP430] Fix data layout string.

2017-06-23 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL306161: [MSP430] Fix data layout string. (authored by dambrouski). Changed prior to commit: https://reviews.llvm.org/D34562?vs=103751&id=103771#toc Repository: rL LLVM https://reviews.llvm.org/D3456

[PATCH] D34562: [MSP430] Fix data layout string.

2017-06-23 Thread Vadzim Dambrouski via Phabricator via cfe-commits
pftbest added a comment. Thanks! > shall we support only EABI or... 2 different ABIs? GCC supports only EABI, and i think we should also spend our limited resources on more important issues, than supporting old abi. https://reviews.llvm.org/D34562 __

[PATCH] D34562: [MSP430] Fix data layout string.

2017-06-23 Thread Anton Korobeynikov via Phabricator via cfe-commits
asl added a comment. Meta-question: shall we support only EABI or... 2 different ABIs? It seems it does not make any sense to support anything besides EABI. https://reviews.llvm.org/D34562 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D34562: [MSP430] Fix data layout string.

2017-06-23 Thread Vadzim Dambrouski via Phabricator via cfe-commits
pftbest created this revision. Change data layout string so it would be compatible with MSP430 EABI. Depends on https://reviews.llvm.org/D34561 https://reviews.llvm.org/D34562 Files: lib/Basic/Targets.cpp test/CodeGen/target-data.c Index: test/CodeGen/target-data.c ==