clang format off

[MateoLa]

I'm trying to compile Stockfish to WebAssembly with Emscripten.
The browser gives me a "Memory access out of bounds" error.

Debbuging the error I came to this piece of code (https://github.com/oficial-stockfish/src/types.h):

// clang-format off
enum PieceType : std::uint8_t {
    NO_PIECE_TYPE, PAWN, KNIGHT, BISHOP, ROOK, QUEEN, KING,
    ALL_PIECES = 0,
    PIECE_TYPE_NB = 8
};

enum Piece : std::uint8_t {
    NO_PIECE,
    W_PAWN = PAWN,     W_KNIGHT, W_BISHOP, W_ROOK, W_QUEEN, W_KING,
    B_PAWN = PAWN + 8, B_KNIGHT, B_BISHOP, B_ROOK, B_QUEEN, B_KING,
    PIECE_NB = 16
};
// clang-format on

To verify the enum I wrote:

std::string getPieceType(PieceType pt) {
    switch (pt) {
        case ALL_PIECES: return "AllPieces";
        case NO_PIECE_TYPE: return "NoPieceType";
        case PAWN: return "Pawn";
        case KNIGHT: return "Knight";
        case BISHOP: return "Bishop";
        case ROOK: return "Rook";
        case QUEEN: return "Queen";
        case KING: return "King";
        case PIECE_TYPE_NB: return "Sentinel";
    }
}

And the compiler output is:

emscripten/mateola.cpp:12:14: error: duplicate case value: 'ALL_PIECES' and 'NO_PIECE_TYPE' both
      equal '0'
   12 |         case NO_PIECE_TYPE: return "NoPieceType";
      |              ^
emscripten/mateola.cpp:11:14: note: previous case defined here
   11 |         case ALL_PIECES: return "AllPieces";
      |              ^
1 error generated.

I think this is the source of the error.
I can't refactor the enum because the program use this structure for bitmaps and nnue calculations.
I know Emscripten use EM_ASM to instruct the formatter to ignore blocks of code with commets but its says nothing about block of codes with code.

Do you have any suggestion to find a solution?

This is related to llvm/llvm-project#106983 issue.
I didn't want to open this as an issue becouse I think this goes beyond of that.
I can say that chess.com and lichess.org are using stockfish18 (I don't know if they do it in their web versions).

[juj]

I don't think the issue has to do with the clang-format on/off.

Try replacing

        case ALL_PIECES: return "AllPieces";
        case NO_PIECE_TYPE: return "NoPieceType";

with

//        case ALL_PIECES: return "AllPieces";
        case NO_PIECE_TYPE: return "AllPieces/NoPieceType";

Then it should build.

[MateoLa]

mmm, you are right, this works. Thank you