fix #14034: AST broken for 'or' keyword

[ludviggunne]

No description provided.

[danmar]

In C it's not a keyword. It's a macro defined in iso646.h. So ideally we should be a bit careful.
In C++ I believe we can treat it as a keyword.
In C it would be good to check if iso646 is included or if the token is used as an operator or variable name..

[ludviggunne]

In C it's not a keyword. It's a macro defined in iso646.h. So ideally we should be a bit careful. In C++ I believe we can treat it as a keyword. In C it would be good to check if iso646 is included or if the token is used as an operator or variable name..

I believe the current approach is to always simplify it if it's used as an operator. I think it's fine without looking for the <iso646.h> since the code wouldn't compile otherwise anyway.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[danmar]

I believe the current approach is to always simplify it if it's used as an operator.

ah yes.. I looked too quickly I thought you were doing something in the AST. This looks fine however there is some unhappy CI.