phosek added a comment.
This was discussed when `LLVM_ENABLE_PER_TARGET_RUNTIME_DIR` was introduced and
there was a pushback against changing the driver behavior depending on the
value of that option, so if we're going to reverse that decision for BareMetal,
I think that deserves a broader discussion.
================
Comment at: clang/lib/Driver/ToolChains/BareMetal.cpp:163
+ CmdArgs.push_back(
+ Args.MakeArgString("-lclang_rt.builtins-" + getTriple().getArchName()));
+#endif
----------------
Is there a reason why BareMetal doesn't just use
https://github.com/llvm/llvm-project/blob/master/clang/lib/Driver/ToolChain.cpp#L462
like all other drivers, and instead re-implements the runtime library lookup
logic?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86877/new/
https://reviews.llvm.org/D86877
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
