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

Kristina Brooks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 10 18:01:11 PDT 2018


kristina added a comment.

Sorry, didn't notice this was recently added, from looking at Darwin and Android embedded tests, this doesn't really match up with either of them. The hosted tests for Android using `libbuiltins` look like this:

  // RUN: %clang -target arm-linux-androideabi -rtlib=compiler-rt -### %s 2>&1 | FileCheck %s -check-prefix ARM-ANDROID
  // ARM-ANDROID: "{{.*[/\\]}}libclang_rt.builtins-arm-android.a"

Not sure what to make of this to be honest, I don't think this should be passed in as a conventional library since the driver usually picks the correct runtime for a given target triple.


https://reviews.llvm.org/D51899





More information about the cfe-commits mailing list