zahiraam wrote:

> Also, if at all possible, I think we only want the extra field for "library 
> builtins" and not all builtins.

Yes, I have tried limiting it to `LibBuiltin` but the addition of 
`FrexpF16F128` led me to add it `Builtin` too.

> Alternatively, we don't need tablegen changes if we leave the encoding of 
> which language standard a builtin is constexpr since to a helper function.

Will look into this.

Thanks.



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

Reply via email to