<div dir="ltr">This question is probably more fit for cfe-dev (CC'd). I'd guess that it has to do with which declaration we look at when determining the function attributes.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 25, 2015 at 5:32 PM, Balaram Makam <span dir="ltr"><<a href="mailto:bmakam@codeaurora.org" target="_blank">bmakam@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="#0563C1" vlink="#954F72"><div><p class="MsoNormal"><span style="color:#1f497d">ping<u></u><u></u></span></p><p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p><div><div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in"><p class="MsoNormal"><b>From:</b> Balaram Makam [mailto:<a href="mailto:bmakam@codeaurora.org" target="_blank">bmakam@codeaurora.org</a>] <br><b>Sent:</b> Wednesday, March 25, 2015 4:43 PM<br><b>To:</b> '<a href="mailto:llvmdev@cs.uiuc.edu" target="_blank">llvmdev@cs.uiuc.edu</a>'<br><b>Subject:</b> [Inliner] passing inline hint to optimizer if it appears explicitly on declaration<u></u><u></u></p></div></div><div><div class="h5"><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">All,<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">I explicitly pass inline hint to the function declaration of foo<int>::add() as in the following example:<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">#cat example.cpp<u></u><u></u></p><p class="MsoNormal">template <class T> class foo {<u></u><u></u></p><p class="MsoNormal">  public:<u></u><u></u></p><p class="MsoNormal">    T* tp;<u></u><u></u></p><p class="MsoNormal">    int elem;<u></u><u></u></p><p class="MsoNormal">    int maxelem;<u></u><u></u></p><p class="MsoNormal">    T* add();<u></u><u></u></p><p class="MsoNormal">};<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">template <class T> inline T* foo <T>::add()<u></u><u></u></p><p class="MsoNormal">{<u></u><u></u></p><p class="MsoNormal">  if (elem!=maxelem)<u></u><u></u></p><p class="MsoNormal">    elem++;<u></u><u></u></p><p class="MsoNormal">  return tp+elem-1;<u></u><u></u></p><p class="MsoNormal">}<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">typedef foo <int> intboundart;<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">class bar<u></u><u></u></p><p class="MsoNormal">{<u></u><u></u></p><p class="MsoNormal">  void addtobound(intboundart& boundarp);<u></u><u></u></p><p class="MsoNormal">};<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">void bar::addtobound(intboundart& barp)<u></u><u></u></p><p class="MsoNormal">{<u></u><u></u></p><p class="MsoNormal">  barp.add();<u></u><u></u></p><p class="MsoNormal">}<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Yet the IR passed into optimizer doesn’t contain inlinehint attribute:<u></u><u></u></p><p class="MsoNormal">#clang -cc1 -emit-llvm –o - example.cpp<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><snip><u></u><u></u></p><p class="MsoNormal">; Function Attrs: nounwind<u></u><u></u></p><p class="MsoNormal">define linkonce_odr i32* @_ZN3fooIiE3addEv(%class.foo* %this) #0 comdat align 2 {<u></u><u></u></p><p class="MsoNormal"><snip><u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Clang will pass inline hint to optimizer only if I declare explicitly in the class like this:<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">template <class T> class foo {<u></u><u></u></p><p class="MsoNormal">  public:<u></u><u></u></p><p class="MsoNormal">    T* tp;<u></u><u></u></p><p class="MsoNormal">    int elem;<u></u><u></u></p><p class="MsoNormal">    int maxelem;<u></u><u></u></p><p class="MsoNormal">    inline T* add();<u></u><u></u></p><p class="MsoNormal">};<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Isn’t this a clang bug? Any pointers on how to fix this?<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Thanks,<u></u><u></u></p><p class="MsoNormal">Balaram<u></u><u></u></p></div></div></div></div><br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br></div>