[PATCH] D51899: Remove extraneous ".a" suffix from baremetal clang_rt.builtins when compiling for baremetal.

2018-09-10 Thread George Morgan via Phabricator via cfe-commits
georgemorgan added a comment. @kristina, the library isn't being passed as an object file to the linker but rather as a library name. Hence, the `.a` suffix is not correct. https://reviews.llvm.org/D51899 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D51899: Remove extraneous ".a" suffix from baremetal clang_rt.builtins when compiling for baremetal.

2018-09-10 Thread George Morgan via Phabricator via cfe-commits
georgemorgan updated this revision to Diff 164776. georgemorgan added a comment. Update diff to provide context. https://reviews.llvm.org/D51899 Files: lib/Driver/ToolChains/BareMetal.cpp test/Driver/baremetal.cpp Index: test/Driver/baremetal.cpp ==

[PATCH] D51899: Remove extraneous ".a" suffix from baremetal clang_rt.builtins when compiling for baremetal.

2018-09-10 Thread George Morgan via Phabricator via cfe-commits
georgemorgan created this revision. georgemorgan added reviewers: joker-eph-DISABLED, jroelofs, harlanhaskins, CodaFi. Herald added a reviewer: javed.absar. Herald added subscribers: cfe-commits, kristina, kristof.beyls. When building for a baremetal triple an extraneous ".a" is appended to the