This is a very new pass, and I don't have every version of GCC. I'm still cleaning up a few warnings etc. Please hang in there for a little bit longer, lots of fixes are in flight for me, but waiting on my builds to finish to make sure they're correct.<div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Sep 14, 2012 at 3:11 AM, Florian Weimer <span dir="ltr"><<a href="mailto:fweimer@redhat.com" target="_blank">fweimer@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 09/14/2012 12:09 PM, NAKAMURA Takumi wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2012/9/14 Florian Weimer <<a href="mailto:fweimer@redhat.com" target="_blank">fweimer@redhat.com</a>>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 09/14/2012 11:55 AM, NAKAMURA Takumi wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-  void dump(const_iterator I) const LLVM_ATTRIBUTE_NOINLINE<br>
LLVM_ATTRIBUTE_USED;<br>
-  void dump() const LLVM_ATTRIBUTE_NOINLINE LLVM_ATTRIBUTE_USED;<br>
+  void dump(const_iterator I) LLVM_ATTRIBUTE_NOINLINE LLVM_ATTRIBUTE_USED<br>
const;<br>
+  void dump() LLVM_ATTRIBUTE_NOINLINE LLVM_ATTRIBUTE_USED const;<br>
</blockquote>
<br>
<br>
This breaks compilation with GCC 4.7.  I don't think GCC supports multiple<br>
__attribute__s per declaration.<br>
</blockquote>
<br>
Florian, I believe r163886 could appease gcc.<br>
</blockquote>
<br></div>
Indeed it does, thanks.<br>
<br>
FYI, GCC reports two warnings, not sure if they are appropriate:<br>
<br>
../lib/Transforms/Scalar/SROA.<u></u>cpp: In member function ‘bool {anonymous}::<u></u>AllocaPartitionRewriter::<u></u>visitMemTransferInst(llvm::<u></u>MemTransferInst&)’:<br>
../lib/Transforms/Scalar/SROA.<u></u>cpp:1901:16: warning: unused variable ‘OrigBegin’ [-Wunused-variable]<br>
../lib/Transforms/Scalar/SROA.<u></u>cpp: In member function ‘bool {anonymous}::<u></u>AllocaPartitionRewriter::<u></u>visitIntrinsicInst(llvm::<u></u>IntrinsicInst&)’:<br>
../lib/Transforms/Scalar/SROA.<u></u>cpp:1997:12: warning: variable ‘New’ set but not used [-Wunused-but-set-variable]<div class="im HOEnZb"><br>
<br>
-- <br>
Florian Weimer / Red Hat Product Security Team<br></div><div class="HOEnZb"><div class="h5">
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>