[cfe-commits] r156531 - in /cfe/trunk: include/clang/Sema/Sema.h lib/Sema/SemaDecl.cpp lib/Sema/SemaDeclAttr.cpp lib/Sema/TargetAttributesSema.cpp test/Index/complete-with-annotations.cpp test/Sema/attr-visibility.c test/Sema/dllimport-dllexport.

[Dmitri Gribenko]

Hi Rafael,

On Thu, May 10, 2012 at 5:50 AM, Rafael Espindola
<rafael.espindola at gmail.com> wrote:
> --- cfe/trunk/lib/Sema/SemaDeclAttr.cpp (original)
> +++ cfe/trunk/lib/Sema/SemaDeclAttr.cpp Wed May  9 21:50:16 2012
[...]
> @@ -3988,11 +4014,9 @@
>                                     bool NonInheritable, bool Inheritable) {
>   SmallVector<const AttributeList*, 4> attrs;
>   for (const AttributeList* l = AttrList; l; l = l->getNext()) {
> +    ProcessDeclAttribute(*this, S, D, *l, NonInheritable, Inheritable);
>     attrs.push_back(l);
>   }
> -  for (int i = attrs.size() - 1; i >= 0; --i) {
> -    ProcessDeclAttribute(*this, S, D, *attrs[i], NonInheritable, Inheritable);
> -  }
[...]

`attrs' variable is effectively unused now.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/