<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Dec 13, 2017, at 8:35 PM, Saleem Abdulrasool &lt;<a href="mailto:compnerd@compnerd.org" class="">compnerd@compnerd.org</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">On Wed, Dec 13, 2017 at 4:14 PM, John McCall <span dir="ltr" class="">&lt;<a href="mailto:rjmccall@apple.com" target="_blank" class="">rjmccall@apple.com</a>&gt;</span> wrote:<br class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><br class=""><div class=""><span class=""><blockquote type="cite" class=""><div class="">On Dec 13, 2017, at 6:42 PM, Saleem Abdulrasool &lt;<a href="mailto:compnerd@compnerd.org" target="_blank" class="">compnerd@compnerd.org</a>&gt; wrote:</div><br class="m_4166797275285532235Apple-interchange-newline"><div class=""><div style="word-wrap:break-word;line-break:after-white-space" class=""><br class=""><div class=""><br class=""><blockquote type="cite" class=""><div class="">On Dec 13, 2017, at 12:46 PM, John McCall &lt;<a href="mailto:rjmccall@apple.com" target="_blank" class="">rjmccall@apple.com</a>&gt; wrote:</div><br class="m_4166797275285532235Apple-interchange-newline"><div class=""><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class=""><blockquote type="cite" class=""><div class=""><br class="m_4166797275285532235Apple-interchange-newline">On Dec 13, 2017, at 3:22 PM, Saleem Abdulrasool &lt;<a href="mailto:compnerd@compnerd.org" target="_blank" class="">compnerd@compnerd.org</a>&gt; wrote:</div><br class="m_4166797275285532235Apple-interchange-newline"><div class=""><div style="word-wrap:break-word;line-break:after-white-space" class=""><br class=""><div class=""><br class=""><blockquote type="cite" class=""><div class="">On Dec 13, 2017, at 12:14 PM, John McCall &lt;<a href="mailto:rjmccall@apple.com" target="_blank" class="">rjmccall@apple.com</a>&gt; wrote:</div><br class="m_4166797275285532235Apple-interchange-newline"><div class=""><blockquote type="cite" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class=""><br class="m_4166797275285532235Apple-interchange-newline">On Dec 13, 2017, at 2:56 PM, Saleem Abdulrasool &lt;<a href="mailto:compnerd@compnerd.org" target="_blank" class="">compnerd@compnerd.org</a>&gt; wrote:<br class=""><br class="">Hey guys,<br class=""><br class="">I have another fun case that things go wrong in :-).&nbsp; We do not preserve the calling convention information when importing function decls via the ClangImporter.&nbsp; I have a pretty simple example in apple/swift#13404.&nbsp; Importing the following:<br class=""><br class=""><blockquote type="cite" class="">float frand(void);<br class="">float fadd(float f, float g) { return f + g; }<br class=""></blockquote><br class="">and using this in swift as:<br class=""><br class=""><blockquote type="cite" class="">func f -&gt; Float { return fadd(frand(), frand()); }<br class=""></blockquote><br class=""><br class="">will result in the fadd call being elided due to the UB in the CC mismatch.&nbsp; Im pretty sure that we should be preserving CC information when importing the interface, probably in `VisitFunctionDecl` in ImportDecl.cpp (although, I believe it can also be lazily computed).&nbsp; Im not sure which really would be the best thing to do here.<br class=""><br class="">This can show up on other targets when interfaces uses `__attribute__((__pcs__))` or `__attribute__((__fastcall__))<wbr class="">`, `__attribute__((__vectorcall__<wbr class="">))`, `__attribute__((__regparm__([<wbr class="">1-3])))`, `__attribute__((__stdcall__))`<wbr class="">, `__attribute__((__thiscall__))<wbr class="">`.<br class=""></blockquote><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class=""><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;float:none;display:inline!important" class="">Without enhancing SIL, I'm not sure we have a good option besides refusing to import function declarations with non-standard CCs.</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class=""><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class=""><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;float:none;display:inline!important" class="">I don't think we really want to change AST function types to handle arbitrary imported calling conventions, but we could change SILFunctionType to be able to store a Clang CC. However, this will require a little bit of extra work in that, if someone tries to pass around the address of a fastcall function as a @convention(c) function, we will have to introduce a thunk.&nbsp; I believe we already do similar kinds of thunking in SILGen, though.</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class=""></div></blockquote><div class=""><br class=""></div><div class="">This is slightly problematic as Linux ARM HF and Windows ARM both use a non-C default (arm_aapcs_vfpcc) which is the test case in the mentioned PR.&nbsp; That is, even without the attributes the declarations above have a non-C CC.</div><div class=""><br class=""></div><div class="">I don't think that we are supporting arbitrary calling conventions per se.&nbsp; This only becomes a problem at the FFI layer, where we want to convert the swift CC to the foreign CC.&nbsp; The IRGen at that point will generate UB which will truncate the implementation when the LLVM optimizer runs.&nbsp; I think that we should at least support AAPCS in both, the standard and VFP, variants at the very least.</div></div></div></div></blockquote><div class=""><br class=""></div></div><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;float:none;display:inline!important" class="">Supporting two C calling conventions is not easier than supporting arbitrary C calling conventions.&nbsp; All the complexity is in how and where we represent those CCs.&nbsp; I'm hesitant to introduce this complexity (and non-portability) into the core language, which is why I'm suggesting just introducing it into SIL.&nbsp; But I explicitly do not want us to hard-code specific non-standard conventions from C outside of maybe attribute parsing; we should generalize the representation to support arbitrary CCs.&nbsp; I don't think this is hard at the SIL level.</span></div></blockquote><div class=""><br class=""></div><div class="">Hmm, I think that we might be looking at different problems.&nbsp; I want to support the de facto standard CC on the target.&nbsp; If you consider Windows ARM, “c” is entirely invalid.</div></div></div></div></blockquote><div class=""><br class=""></div></span><div class="">@convention(c) cannot be "entirely invalid". &nbsp;@convention(c) means the platform's C calling convention, which I promise does always exist.&nbsp; The code "int foo(void); int main() { foo(); }" does actually compile on Windows ARM, and it uses a calling convention; that is the platform's C calling convention.</div></div></div></blockquote><div class=""><br class=""></div><div class="">Ah, I was interpreting it to be equivalent to LLVM's notion of the C calling convention which is not the same as you describe it.&nbsp; On Windows ARM, the C convention requires an additional annotation of the function call.</div><div class="">&nbsp;</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""><span class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap:break-word;line-break:after-white-space" class=""><div class=""><div class="">The equivalent of “c” is arm_aapcs_vfpcc.&nbsp; However, there are other cases where we do need a secondary CC.&nbsp; As an example of that, the AEABI RT function calls are always made as AAPCS, but on Linux ARM HF targets (e.g. armv7-unknown-linux-gnueabihf)<wbr class="">, the default CC is the VFP variant of AAPCS, so we do need a secondary CC there for some library calls.&nbsp; We usually get lucky as the function calls there are formed by the backend which knows the CC necessary for the call there.</div></div></div></div></blockquote><div class=""><br class=""></div></span>I think you might be confusing "the C calling convention" with some other concept — maybe the default LLVM calling convention?&nbsp; I believe it's true that LLVM has backends, notably ARM, where the maintainers made the (IMO questionable) decision to have the default LLVM CC be the same across operating systems instead of considering the full target triple.&nbsp; That just means that frontends have an obligation to set the target's actual CC on every C call and function.&nbsp; I would readily believe that Swift IRGen has bugs — maybe even pervasive ones — where we fail to do that.</div></div></blockquote><div class=""><br class=""></div><div class="">Yes, this is the scenario that I was describing - the need for the frontend to annotate the call with he correct calling convention for the target (which is equivalent to swift's notion of `@convention(c)`).&nbsp; However, the CC annotation is entirely missing even for uses of the declarations as the information is not preserved during the import stage.</div></div></div></div></div></blockquote><div><br class=""></div>It's not in the Clang AST, either. &nbsp;Clang function types only carry a calling convention when they're explicitly using a non-standard convention. &nbsp;This is all fine except that we shouldn't just be ignoring non-standard conventions in the importer.</div><div>&nbsp;<br class=""><blockquote type="cite" class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""><span class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap:break-word;line-break:after-white-space" class=""><div class=""><blockquote type="cite" class=""><div class=""><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class="">In general, the importer has a lot of flexibility when importing declarations and very little flexibility when importing types, especially pointer types.&nbsp; We can only import one kind of function pointer type as a @convention(c) function type; everything else needs to be imported as an opaque pointer unless we actually bite the bullet and enhance the AST @convention system to support more C conventions.&nbsp; For maximum expressivity, that function pointer type needs to be the type most commonly used for function pointers on the platform, assuming there is one.&nbsp; It doesn't really matter if that's not the default calling convention for function *declarations* because we can always pass around a specific variant-CC function as a @convention(c) function value by introducing a thunk; function-pointer equality won't work, but probably nobody cares.&nbsp; What we can't do is turn an arbitrary variant-CC function *pointer* into a @convention(c) function pointer.&nbsp; But the correctness of all this relies on SIL being able to fully represent the C calling convention.</div></div></blockquote><div class=""><br class=""></div><div class="">In the case of support of a target which does not support the C calling convention but expects all calls to be of a specific convention, we currently fall apart.&nbsp; It sounds like you would prefer that the approach to handle that would be to map `@convention(c)` to that alternate calling convention?&nbsp; For importing declarations, we currently do not preserve the calling convention at all.&nbsp; As a result, right now, FFI calls into C may introduce UB in the IR.</div></div></div></div></blockquote><div class=""><br class=""></div></span>Yes, we clearly have a responsibility to call things with the right calling convention, and we should not import types and declarations which we cannot meet that responsibility for.&nbsp; I am trying to describe the correct technical design for that, which is:</div></div></blockquote><div class=""><br class=""></div><div class="">Great, so we agree on this :-).</div><div class="">&nbsp;</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">&nbsp; - In the importer, we do not need to assign a specific convention to the type of an imported function declaration.</div></div></blockquote><div class=""><br class=""></div><div class="">I don't understand how this works when we may need to annotate the function call to match the C calling convention.</div></div></div></div></blockquote><div><br class=""></div>SIL knows how to map a Swift declaration back to a Clang declaration and use its type to guide SIL type lowering. &nbsp;There's a somewhat obscure set of assumptions at work here.</div><div>&nbsp;<br class=""><blockquote type="cite" class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">&nbsp; - In the importer, we should import function pointer types as @convention(c) function types if they use the target's C calling convention and as opaque pointer types otherwise.</div></div></blockquote><div class=""><br class=""></div><div class="">Fair enough.</div><div class="">&nbsp;</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">&nbsp; - SILFunctionType should have the ability to express a calling convention more precisely than just @convention(c).&nbsp; IRGen can just use this value when explicit.&nbsp; For @convention(c), IRGen can ask Clang for the correct LLVM CC to use.</div></div></blockquote><div class=""><br class=""></div><div class="">Agreed that the calls should be able to declare this more precisely.&nbsp; Do we want to round trip back to clang to get the LLVM CC for this call?&nbsp; Especially if this is the "C" CC?</div></div></div></div></blockquote><div><br class=""></div>SILFunctionType should carry the Clang CC. &nbsp;When Swift IRGen asks Clang how to lower a function call/declaration, Clang will tell it what LLVM CC to use; it's one of the fields on CGFunctionInfo.</div><div>&nbsp;<br class=""><blockquote type="cite" class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">&nbsp; - SIL type lowering should assign function declarations their actual C calling convention if they use a non-standard calling convention.</div></div></blockquote><div class=""><br class=""></div><div class="">Yes.</div><div class="">&nbsp;</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">&nbsp; - SILGen may need to introduce thunks when passing around such functions as @convention(c) function values.&nbsp; Code for this already exists in order to allow C functions to be passed around as native function values.</div></div></blockquote><div class=""><br class=""></div><div class="">This sounds pretty good and would broaden the abilities for swift to FFI to existing code.</div></div></div></div></blockquote><div><br class=""></div>Yeah. &nbsp;This should work well as long as there isn't an API that needs to traffic in non-standard function *pointers*.</div><div><br class=""></div><div>John.</div><div><br class=""><blockquote type="cite" class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class="">&nbsp;</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><span class="HOEnZb"><font color="#888888" class=""><div class="">John.</div></font></span><span class=""><div class=""><br class=""></div><div class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap:break-word;line-break:after-white-space" class=""><div class=""><div class="">I think that the FFI to declarations is far more common than the FFI to a function pointer, which is why I am focusing on the imported declaration rather than the imported types.&nbsp; I do agree that supporting additional conventions in the SIL layer would be good for the imported function pointer case.&nbsp; Am I missing something and is the `@convention` used for the imported declarations too?</div><br class=""><blockquote type="cite" class=""><div class=""><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" class="">John.</div></div></blockquote></div></div></div></blockquote></div></span></div></blockquote></div>-- <br class=""><div class="gmail_signature" data-smartmail="gmail_signature">Saleem Abdulrasool<br class="">compnerd (at) compnerd (dot) org</div>
</div></div>
</blockquote></div><br class=""></body></html>