[llvm] r362128 - Reapply: IR: add optional type to 'byval' function parameters

[Tim Northover via llvm-commits]

Hi Mikael,

Thanks for telling me about that error. I’m looking into it now.

> On 5 Jun 2019, at 05:55, Mikael Holmén <mikael.holmen at ericsson.com> wrote:
> I also noticed that if I run clang on a C program with -emit-llvm, then 
> I don't get the optional byval argument in the output ll-file, but if I 
> run with -emit-llvm-bc and then use llvm-dis on the bc-file, I do get it.

This is intentional though. The bitcode gets auto-upgraded to the new form, which is generally what we do when we change the IR. It should be a very temporary state, I’m writing a Clang patch so that it emits byval with a type.

Cheers.

Tim.