<div dir="ltr">The last time we discussed this, this test case came up:<div><br></div><div><div style="font-family:arial,sans-serif;font-size:13px">  char alignas(A, B) buffer[max(sizeof(A), sizeof(B))];</div><div style="font-family:arial,sans-serif;font-size:13px">
  A *a = reinterpret_cast<A*>(buffer);</div><div style="font-family:arial,sans-serif;font-size:13px">  B *b = reinterpret_cast<B*>(buffer);<br></div><div style="font-family:arial,sans-serif;font-size:13px">  new(buffer) A;</div>
<div style="font-family:arial,sans-serif;font-size:13px">  a->vfn();</div><div style="font-family:arial,sans-serif;font-size:13px">  new(buffer) B;</div><div style="font-family:arial,sans-serif;font-size:13px">  b->vfn();</div>
</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Imagine those placement news are buried in an external function call.</div><div style="font-family:arial,sans-serif;font-size:13px">
<br></div><div style="font-family:arial,sans-serif;font-size:13px">Applying your transformation of adding stores breaks the program, if I'm not mistaken:</div><div style="font-family:arial,sans-serif;font-size:13px"><br>
</div><div style="font-family:arial,sans-serif;font-size:13px"><div>  char alignas(A, B) buffer[max(sizeof(A), sizeof(B))];</div><div>  A *a = reinterpret_cast<A*>(buffer);</div><div>  B *b = reinterpret_cast<B*>(buffer);<br>
</div><div>  new(buffer) A;</div><div>  void *vptr = a->vptr;</div><div>  a->vfn();</div><div>  new(buffer) B;</div><div>  a->vptr = vptr; // Nope, we changed it.  a is dead, long live b.<br></div><div>  b->vfn();</div>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 23, 2014 at 9:02 AM, Matthieu Monrocq <span dir="ltr"><<a href="mailto:matthieu.monrocq@gmail.com" target="_blank">matthieu.monrocq@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>TL;DR: where I wonder about the viability of a seemingly simple change to the code generation in Clang that seem to trick LLVM in doing a lot more devirtualization work than it's doing today... without any change to LLVM itself.<br>

<br></div><div><br>Hello,<br><br></div>I was reading Jan Hubicka's serie of articles on devirtualization in gcc, of which I posted the latest opus to Reddit [1] where you can find links to the 5 parts.<br><br></div>In Part 2, Jan brings up a discussion on llvm-dev [2] where were debated the best ways to bring devirtualizations in the backend, for example in the presence of opaque functions:<br>

<br></div>#include <cstdio><br><br>struct Base { virtual void virtualCall() = 0; };<br>struct Derived: Base { virtual void virtualCall() override { printf("Hello, World!\n"); } };<br><br>void inlinable(Base& b) { b.virtualCall(); }<br>

void opaque(Base& b);<br><br>void func() {<br>    Derived d;<br><br>    inlinable(d); // statement A<br>    opaque(d); // Unknown implementation, LLVM assumes it may change the virtual pointer<br><br>    inlinable(d); // statement B<br>

}<br><div><br></div><div>Now, the difference of treatment between the statement A and the statement B is that, after inlining, LLVM will de-virtualize the call in A but not in B, as can be seen thanks to Coliru [3]:<br><br>

define void @_Z4funcv() #0 {<br>  ; Derived d;<br>  %d = alloca %struct.Derived, align 8<br>  %1 = getelementptr inbounds %struct.Derived* %d, i64 0, i32 0, i32 0<br>  store i32 (...)** bitcast (i8** getelementptr inbounds ([3 x i8*]* @_ZTV7Derived, i64 0, i64 2) to i32 (...)**), i32 (...)*** %1, align 8, !tbaa !1<br>

  %2 = getelementptr inbounds %struct.Derived* %d, i64 0, i32 0<br>  %3 = bitcast %struct.Derived* %d to void (%struct.Base*)***<br><br>  ; inlinable(d); // statement A<br>  %puts.i = call i32 @puts(i8* getelementptr inbounds ([14 x i8]* @str, i64 0, i64 0))<br>

<br>  ; opaque(d);<br>  call void @_Z6opaqueR4Base(%struct.Base* %2)<br><br>  ; inlinable(d); // statement B<br>  %4 = load void (%struct.Base*)*** %3, align 8, !tbaa !1<br>  %5 = load void (%struct.Base*)** %4, align 8<br>

  call void %5(%struct.Base* %2)<br><br>  ret void<br>}<br><br></div><div><div><br></div><div>This is typical of the rift between the C++-aware front-end (Clang) and the C++-unaware middle-end (LLVM):<br><br></div><div>- Clang knows that C++ does not allow a change of dynamic type, and thus of v-ptr, during the lifetime of an object (after construction ended and before destruction begins); but cannot do anything because it knows not how to inline<br>

<br></div><div>- LLVM knows how to inline but is unaware of C++ rules and that those obscure bitcasts are all about virtual calls<br><br><br></div><div>Now, as mentioned in [2] the discussion on how to make LLVM more "aware" of what is going on is complicated; I have seen several proposals over time, the merit of various intrinsics being discussed and no approach being taken: it's a hard problem.<br>

<br><br></div><div>In this very special case, however, it seems to me that Clang could reach out to LLVM. Suppose that Clang introduced a simple store instruction right after the "opaque" call ?<br><br></div><div>

If we simulate the Itanium ABI using a more C-ish approach [4]:<br><br>#include <cstdio><br><br>struct Base;<br><br>struct BaseVTableLayout {<br>    void (*virtualCall)(Base*);<br>};<br><br>struct Base { BaseVTableLayout const* vptr; };<br>

struct Derived: Base {};<br><br>void Derived_virtualCall(Base*) { printf("Hello, World!\n"); }<br><br>BaseVTableLayout const BaseVTable = { 0 };<br>BaseVTableLayout const DerivedVTable = { &Derived_virtualCall };<br>

<br>void inlinable(Base* b) { b->vptr->virtualCall(b); }<br><br>void opaque(Base* b);<br><br>void func() {<br>    Derived d;<br>    d.vptr = &DerivedVTable;<br><br>    inlinable(&d); // statement A<br><br>    opaque(&d); // Unknown implementation, LLVM assumes it may change the virtual pointer<br>

<br>    inlinable(&d); // statement B<br>}<br><br><br></div><div>We see that the exact same behaviour is exhibited by LLVM: in fact the IR is nearly a carbon-copy, except for the name of the globals.<br><br></div><div>

In this C-ish approach, though, introducing the "redundant" store is dead-easy. In fact, we can even be completely naive about it:<br><br>void func() {<br>    Derived d;<br>    d.vptr = &DerivedVTable;<br><br>

    inlinable(&d); // statement A<br>    d.vptr = &DerivedVTable; // inform LLVM that the function is not allowed to change the virtual pointer under our feet<br><br>    opaque(&d); // Unknown implementation, LLVM assumes it may change the virtual pointer<br>

    d.vptr = &DerivedVTable; // inform LLVM that the function is not allowed to change the virtual pointer under our feet<br><br>    inlinable(&d); // statement B<br>    d.vptr = &DerivedVTable; // inform LLVM that the function is not allowed to change the virtual pointer under our feet<br>

}<br><br><br></div><div>And let's look at what LLVM generates:<br><br>define void @_Z4funcv() #1 {<br>  ; Derived d;<br>  %d = alloca %struct.Base, align 8<br>  %1 = getelementptr inbounds %struct.Base* %d, i64 0, i32 0<br>

<br>  ; inlinable(d); // statement A<br>  %puts.i = call i32 @puts(i8* getelementptr inbounds ([14 x i8]* @str, i64 0, i64 0)) #3<br><br>  ; opaque(d);<br>  store %struct.BaseVTableLayout* bitcast ({ void (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %1, align 8, !tbaa !1<br>

<br>  call void @_Z6opaqueP4Base(%struct.Base* %d)<br><br>  store %struct.BaseVTableLayout* bitcast ({ void (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %1, align 8, !tbaa !1<br>

<br>  ; inlinable(d); // statement B<br>  %puts.i1 = call i32 @puts(i8* getelementptr inbounds ([14 x i8]* @str, i64 0, i64 0)) #3<br><br>  ret void<br><br>}<br><br><br></div><div>Bingo! The indirect virtual call in statement B has been inlined!<br>

<br><br></div><div>But let's not stop there. There are plenty of occasions where Clang does not know the dynamic type and yet that can benefit from being simple-minded. Suppose that we start from:<br><br>void funky(Base* b) {<br>

    inlinable(b); // statement A<br><br>    opaque(b); // Unknown implementation, LLVM assumes it may change the virtual pointer<br><br>    inlinable(b); // statement B<br>}<br><br>void func() {<br>    Derived d;<br>    d.vptr = &DerivedVTable;<br>

<br>    funky(&d);<br>}<br><br><br></div><div>Then let's apply this "the dynamic type will not change" rule [5]:<br><br>void funky(Base* b) {<br>    BaseVTableLayout const* vptr = b->vptr;<br>    inlinable(b); // statement A<br>

    b->vptr = vptr; // inform LLVM that the function is not allowed to change the virtual pointer under our feet<br><br>    opaque(b); // Unknown implementation, LLVM assumes it may change the virtual pointer<br>    b->vptr = vptr; // inform LLVM that the function is not allowed to change the virtual pointer under our feet<br>

<br>    inlinable(b); // statement B<br>    b->vptr = vptr; // inform LLVM that the function is not allowed to change the virtual pointer under our feet<br>}<br><br>void func() {<br>    Derived d;<br>    d.vptr = &DerivedVTable;<br>

<br>    funky(&d);<br>}<br></div><div><br><br></div><div>And behold:<br><br>define void @_Z4funcv() #1 {<br>  ; Derived d;<br>  %d = alloca %struct.Base, align 8<br>  %1 = getelementptr inbounds %struct.Base* %d, i64 0, i32 0<br>

<br>  ; inlinable(d); // statement A<br>  %puts.i = call i32 @puts(i8* getelementptr inbounds ([14 x i8]* @str, i64 0, i64 0)) #3<br><br>  ; opaque(d);<br>  store %struct.BaseVTableLayout* bitcast ({ void (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %1, align 8, !tbaa !1<br>

<br>  call void @_Z6opaqueP4Base(%struct.Base* %d)<br><br>  store %struct.BaseVTableLayout* bitcast ({ void (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %1, align 8, !tbaa !1<br>

<br>  ; inlinable(d); // statement B<br>  %puts.i1 = call i32 @puts(i8* getelementptr inbounds ([14 x i8]* @str, i64 0, i64 0)) #3<br><br>  ret void<br>}<br><br></div><div>For those wondering, in the back row, yes that is the exact same LLVM IR than before; it just saw throw our the call to "funky".<br>

<br><br></div><div>And for more fun, trying it in a loop:<br><br>int add(Base** array, size_t size) {<br>    int total = 0;<br>    for (size_t i = 0; i != size; ++i) {<br>        Base* b = array[i];<br>        <br>        opaque(b);<br>

        <br>        total += b->vptr->virtualCall(b);<br>    }<br>    return total;<br>}<br><br>int func() {<br>    Derived d0;<br>    d0.vptr = &DerivedVTable;<br>    Derived d1;<br>    d1.vptr = &DerivedVTable;<br>

    Derived d2;<br>    d2.vptr = &DerivedVTable;<br>    <br>    Base* b[3] = { &d0, &d1, &d2 };<br><br>    return add(b, 3);<br>}<br><br></div><div>The IR looks ugly [6]: "add" is inlined in "func" and the loop is unrolled, but LLVM does not know that "opaque" cannot change the v-ptr, and thus a lot of virtual calls ensue. Fun fact: I started by forgetting to put the call to "opaque" and LLVM directly optimized "func" to "ret 126".<br>

<br></div><div>So, let's apply our simple "the dynamic type will not change" rule like before [7]:<br><br>int add(Base** array, size_t size) {<br>    int total = 0;<br>    for (size_t i = 0; i != size; ++i) {<br>

        Base* b = array[i];<br>        BaseVTableLayout const* vptr = b->vptr;<br>        <br>        opaque(b);<br>        b->vptr = vptr;<br>        <br>        total += b->vptr->virtualCall(b);<br>        b->vptr = vptr;<br>

    }<br>    return total;<br>}<br><br><br></div><div>And now I dare present the IR:<br><br>define i32 @_Z4funcv() #1 {<br>.lr.ph.i:<br>  %d0 = alloca %struct.Base, align 8<br>  %d1 = alloca %struct.Base, align 8<br>  %d2 = alloca %struct.Base, align 8<br>

<br>  %0 = getelementptr inbounds %struct.Base* %d0, i64 0, i32 0<br><br>  store %struct.BaseVTableLayout* bitcast ({ i32 (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %0, align 8, !tbaa !5<br>

<br>  %1 = getelementptr inbounds %struct.Base* %d1, i64 0, i32 0<br><br>  store %struct.BaseVTableLayout* bitcast ({ i32 (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %1, align 8, !tbaa !5<br>

<br>  %2 = getelementptr inbounds %struct.Base* %d2, i64 0, i32 0<br><br>  store %struct.BaseVTableLayout* bitcast ({ i32 (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %2, align 8, !tbaa !5<br>

<br>  call void @_Z6opaqueP4Base(%struct.Base* %d0)<br><br>  store %struct.BaseVTableLayout* bitcast ({ i32 (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %0, align 8, !tbaa !5<br>

<br>  call void @_Z6opaqueP4Base(%struct.Base* %d1)<br><br>  store %struct.BaseVTableLayout* bitcast ({ i32 (%struct.Base*)* }* @_ZL13DerivedVTable to %struct.BaseVTableLayout*), %struct.BaseVTableLayout** %1, align 8, !tbaa !5<br>

<br>  call void @_Z6opaqueP4Base(%struct.Base* %d2)<br><br>  ret i32 126<br>}<br><br></div><div>Where one note that the additions of 42 were folded together.<br></div><div><br><br><br></div><div>Conclusion/Digest:<br></div>

<div><br></div><div>The attempt to implement devirtualization in CodeGenFunction::CanDevirtualizeMemberFunctionCall only get us so far. It seems reasonable to take advantage of the "final" keyword and other language-level constructs, but Clang does not (and should not) implement inlining and constant propagation, and thus does not cross the bridge. Meta-data proposals are still pending, and I still have hope for them, in the mean time though a stop-gap alternative seems interesting.<br>

<br></div><div>A simple addition to the CodeGen process, expressing the rule **the dynamic type of an object is not changed by a function call** in LLVM IR instructions (caching the vptr of an instance as soon as it comes into being and restoring it after each function call involving this instance), seems to be sufficient to enable the devirtualization of calls by LLVM whenever inlining and constant propagation make it possible.<br>

<br></div><div>The only downside that I could fathom are the redundant store instructions: they are the witnesses that LLVM did not consider them redundant and therefore could not have devirtualized calls without them. Proper meta-data would solve this issue I suppose.<br>

<br>Still, in my naivete, I believe those redundant stores to cost less than the devirtualization opportunities currently being missed.<br></div><div><br><br></div><div>I would be very interested in hearing the community feedback on this.<br>

</div><div><br></div><div><br><br> [1] <a href="http://www.reddit.com/r/cpp/comments/23rjda/honza_hubi%C4%8Dkas_blog_devirtualization_in_c/" target="_blank">http://www.reddit.com/r/cpp/comments/23rjda/honza_hubi%C4%8Dkas_blog_devirtualization_in_c/</a><br>

 [2] <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2011-December/046072.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2011-December/046072.html</a><br> [3] <a href="http://coliru.stacked-crooked.com/a/0dc6a12cb5f27543" target="_blank">http://coliru.stacked-crooked.com/a/0dc6a12cb5f27543</a><br>

 [4] <a href="http://coliru.stacked-crooked.com/a/705f607005017c43" target="_blank">http://coliru.stacked-crooked.com/a/705f607005017c43</a><br> [5] <a href="http://coliru.stacked-crooked.com/a/a34b6674255ad66c" target="_blank">http://coliru.stacked-crooked.com/a/a34b6674255ad66c</a><br>

 [6] <a href="http://coliru.stacked-crooked.com/a/93a493d80e7bcc3b" target="_blank">http://coliru.stacked-crooked.com/a/93a493d80e7bcc3b</a><br> [7] <a href="http://coliru.stacked-crooked.com/a/0f0d8470c1b066d0" target="_blank">http://coliru.stacked-crooked.com/a/0f0d8470c1b066d0</a><br>

</div></div></div>
<br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>