[PATCH] D65699: [Driver] Prioritize SYSROOT/usr/include over RESOURCE_DIR/include on linux-musl

Khem Raj via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 8 23:20:14 PDT 2019


raj.khem added a comment.

one issue I now see is that when we have some thing like

static const struct {

          const wchar_t *name;
          int (*func)(EditLine *, int, const wchar_t **);
  } cmds[] = {
          { L"bind",              map_bind        },
          { L"echotc",            terminal_echotc },
          { L"edit",              el_editmode     },
          { L"history",           hist_command    },
          { L"telltc",            terminal_telltc },
          { L"settc",             terminal_settc  },
          { L"setty",             tty_stty        },
          { NULL,                 NULL            }
  };

clang complains

warning: incompatible pointer types initializing 'const wchar_t *' (aka 'const long *') with an expression of type 'int [5]' [-Wincompatible-pointer-types]

  { L"bind",              map_bind        },
    ^~~~~~~

it seems L"..." prefix is treated as `int`  where as in musl wchar_t is of long int type


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65699/new/

https://reviews.llvm.org/D65699





More information about the cfe-commits mailing list