================
@@ -6343,11 +6343,16 @@ namespace {
};
} // end anonymous namespace
-static void
-fillDependentAddressSpaceTypeLoc(DependentAddressSpaceTypeLoc DASTL,
- const ParsedAttributesView &Attrs) {
- for (const ParsedAttr &AL : Attrs) {
- if (AL.getKind() == ParsedAttr::AT_AddressSpace) {
+static void fillDependentAddressSpaceTypeLoc(
+ DependentAddressSpaceTypeLoc DASTL,
+ ArrayRef<const ParsedAttributesView *> AttrLists) {
+ for (const ParsedAttributesView *Attrs : AttrLists) {
+ for (const ParsedAttr &AL : *Attrs) {
+ if (AL.getKind() != ParsedAttr::AT_AddressSpace)
+ continue;
+ // Skip invalid or malformed attributes; they did not produce a type.
+ if (AL.isInvalid() || AL.getNumArgs() != 1 || !AL.isArgExpr(0))
+ continue;
----------------
TPPPP72 wrote:
Let's combine it to a single `if` statement.
https://github.com/llvm/llvm-project/pull/216348
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
