[llvm-dev] Passing arguments to var args function

Alex Bradbury via llvm-dev llvm-dev at lists.llvm.org
Wed Aug 8 07:59:45 PDT 2018


On 8 August 2018 at 14:49, Tim Northover via llvm-dev
<llvm-dev at lists.llvm.org> wrote:
> On Wed, 8 Aug 2018 at 14:42, Liad Mordekoviz
> <liad.mordekoviz at nextsilicon.com> wrote:
>> I was wondering how the community would accept a change in the TD,
>> if the code at CallinvConvLower.cpp:138 would've passed the IsFixed flag to the AssignFn then all those workarounds wouldn't be needed,
>
> I think that would be a pretty good idea. I suspect the main reason it
> wasn't done for AArch64 is that the backend was secret at the time and
> Jim wanted to minimize merge conflicts.

Anyone looking at this issue may want to take a look at my patch here
https://reviews.llvm.org/D42374 which adds an IsFixed field to
ISD::ArgFlagsTy.

Best,

Alex


More information about the llvm-dev mailing list