[PATCH] D116110: Introduce the AttributeMask class

Nikita Popov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 22 01:08:17 PST 2021


nikic added a comment.

Looking at some of the changes here, I landed a couple of cleanup commits:

- https://github.com/llvm/llvm-project/commit/3b0f5a4856fce76a6535feddd1692747b81b60cd (this should fix the build failure)
- https://github.com/llvm/llvm-project/commit/e8e8bfeeb7233bde17d0a811b87d392271b33f42 (unnecessary AttrBuilder use for one attribute)
- https://github.com/llvm/llvm-project/commit/9f24f010abe6b9db3d3fd913805c9e0779d00334 (make RS4GC attribute removal more idiomatic)

I think the latter change should allow you to drop the `AttrMask::hasAttributes()` method from the API.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116110/new/

https://reviews.llvm.org/D116110



More information about the cfe-commits mailing list