[PATCH] D43248: [Attr] Fix parameter indexing for attributes
Joel E. Denny via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 13 15:27:09 PDT 2018
jdenny added a comment.
In https://reviews.llvm.org/D43248#1036439, @jdenny wrote:
> So, I'm planning to remove test/Frontend/ast-attr.cpp, rename test/Sema/attr-print.cpp to test/Misc/attr-print-emit.cpp, and change its run lines to:
>
> // RUN: %clang_cc1 %s -ast-print | FileCheck %s
> // RUN: %clang -emit-ast -o %t.ast %s
> // RUN: %clang_cc1 %t.ast -ast-print | FileCheck %s
>
>
> That seems to work fine locally. I'll just leave the old test/Sema/attr-print.c alone as it's not part of this problem. Any objections?
I went ahead and committed. It's in r327456.
Repository:
rC Clang
https://reviews.llvm.org/D43248
More information about the cfe-commits
mailing list