aaron.ballman added inline comments.
================ Comment at: clang/include/clang/AST/Attr.h:346 + Tune == Other.Tune && + BranchProtection == Other.BranchProtection && + Features == Other.Features; ---------------- craig.topper wrote: > Was it a bug that BranchProtection wasn't in the original code? I just dug out the original review which added it (https://reviews.llvm.org/D68711) and there's no mention of why `BranchProtection` was missed here, so I think it was just a bug. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86187/new/ https://reviews.llvm.org/D86187 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
