mikerice1969 wrote:

The old tablegen produced a single entry per name:

`.Case("unroll", true)`

The new tablegen can produce multiple entries:

```
.Case("Microsoft::unroll", true)
.Case("Pragma::unroll", true)
```

I think we can instead produce something like this, one per name:

```
.Case("unroll",
(Syntax==AttributeCommonInfo::AS_Microsoft && !Scope) || 
(Syntax==AttributeCommonInfo::AS_Pragma && Scope && Scope=="whatever") ||
...)
```
That should elimate the need for any new std::strings.  @AaronBallman does this 
sound reasonable?

https://github.com/llvm/llvm-project/pull/99993
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to