erik.pilkington marked an inline comment as done.
erik.pilkington added inline comments.
================
Comment at: clang/test/FixIt/fixit-pragma-attribute.cpp:19-20
#pragma clang attribute push
(__attribute__((annotate("subRuleContradictions"))), apply_to = any(variable,
variable(is_parameter), function(is_member), variable(is_global)))
-// CHECK: fix-it:{{.*}}:{[[@LINE-1]]:108-[[@LINE-1]]:132}:""
-// CHECK: fix-it:{{.*}}:{[[@LINE-2]]:153-[[@LINE-2]]:172}:""
+// CHECK: fix-it:{{.*}}:{[[@LINE-1]]:153-[[@LINE-1]]:172}:""
+// CHECK: fix-it:{{.*}}:{[[@LINE-2]]:108-[[@LINE-2]]:132}:""
----------------
aaron.ballman wrote:
> Any idea why this swapped?
Its because I added a new subject match rule in Attr.td, and the match rules
enumerators are being stored in a DenseMap in
Sema::ActOnPragmaAttributeAttribute. I think we should make that a std::map or
something so this test is less fragile.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60542/new/
https://reviews.llvm.org/D60542
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
