<div dir="ltr"><div><div><div>Put it up for review in Phabricator, with a reference to the bug.<br>For more on phabricator: <a href="http://llvm.org/docs/Phabricator.html">http://llvm.org/docs/Phabricator.html</a><br><br></div>You'll probably be asked to add a test-case so that if someone removes/alters that line, some test(s) will fail. (Using your sampel would probably be fine, just wrapping the usual test-stuff around it - find some similar type of tests somewhere and do suitable copy/paste/modify editing).<br><br></div><div>Wait for feedback on the review, update as appropriate, repeat until approved.<br></div><div><br></div>Once the review is approved, ask someone to commit it (I presume you don't have permission to commit).<br><br>--<br></div><div>Mats<br></div><div><br>--<br></div>Mats<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 12 January 2017 at 07:46, Félix Cloutier via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hello all,<div><br></div><div>About a year ago, I found a bug with the interaction of address spaces and C++ pointers to members. Applying a pointer-to-member to a pointer discards the address space of the pointer if it had one. For instance, with this code:</div><div><br></div><div><div></div><blockquote type="cite"><div>struct foo {</div><div>    int bar;</div><div>};</div><div><br></div><div>#define AS1 __attribute__((address_space(<wbr>1)))</div><div><br></div><div>int main()</div><div>{</div><div>    foo AS1* f;</div><div>    int AS1* x = &(f->*(&foo::bar));</div><div>}</div></blockquote></div><div><br></div><div>Clang says:</div><div><br></div><div><blockquote type="cite">cannot initialize a variable of type '__attribute__((address_space(<wbr>1))) int *' with an rvalue of type 'int *'</blockquote></div><div><br></div><div>I filed a bug (<a href="https://llvm.org/bugs/show_bug.cgi?id=25941" target="_blank">https://llvm.org/bugs/show_<wbr>bug.cgi?id=25941</a>) and even had a patch for it when I found it, but I never followed through. The patch is one single line of code and appears to be still applicable. What are the next steps to get the problem fixed?</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Félix</div></font></span></div><br>______________________________<wbr>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>