[PATCH] D54449: [compiler-rt][builtins][PowerPC] Enable builtins tests on PowerPC 64 bit LE

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added a comment.

In https://reviews.llvm.org/D54449#1296245, @dschuff wrote:

> Conceptually, `COMPILER_RT_ABI` is the calling convention used by all compiler-rt functions. It is explicitly named and used because it may be different from the default calling convention on some targets. In practice I believe it only affects ARM, where the calling convention uses ARM's soft-float ABI (where IIRC floats are passed in integer registers) even on targets that use the hard-float ABI by default. The compiler knows the calling conventions and emits the right code when it emits the libcalls, and the ABI declaration on the implementations causes the right calling convention to be used there. The unusual thing about the tests is that they locally declare the compiler-rt functions explicitly. So this declaration has to match the implementation. It may not matter for PPC but my recommendation would be "just use it everywhere in compiler-rt" for simplicity.


Fair enough, but shouldn't the header used for the tests `#include "lib/builtins/int_lib.h"` in order to have the macro def available?


https://reviews.llvm.org/D54449