[PATCH] D64962: appendToGlobalCtors: allow for llvm.global_ctors functions of varying type
Anthony Eden via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 10 09:46:39 PDT 2020
aeden added a comment.
> Relaxing it to Constant and asserting that it has the right type (pointer to void function) seems reasonable. I would prefer that the caller be the one to insert the cast from the `void(int, char**, char**)` prototype to the void prototype, since only the caller can know that it is safe.
Okay, sure.
> And it would need a test to proceed. A unit test would be best.
I've been really busy recently and haven't had a chance to do this. Sorry!
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64962/new/
https://reviews.llvm.org/D64962
More information about the llvm-commits
mailing list