[llvm-commits] Please review the patch for IntegersSubsetMapping

[Stepan Dyatkovskiy]

Stepan Dyatkovskiy wrote:
> SimplifyCFG pass will optimize two these folded switches to the single one:
>
> switch (c) {
>     case '0'..'1': // BITS_SPECIFIC_SUBSET
>       tryProcessBit(c);
>       break;
>     case '2'..'9': // DECS_SPECIFIC_SUBSET
>       tryProcessDecs(c);
>       break;
>     case 'A'..'F': // HEXS_SPECIFIC_SUBSET
>       tryProcessHexs(c);
>       break;
>     default:
>       processSomethingElse();
> }

Forgot about '\\' character. Fixed simplified switch is below:

switch (c) {
    case '0'..'1': // BITS_SPECIFIC_SUBSET
      tryProcessBit(c);
      break;
    case '2'..'9': // DECS_SPECIFIC_SUBSET
      tryProcessDecs(c);
      break;
    case 'A'..'F': // HEXS_SPECIFIC_SUBSET
      tryProcessHexs(c);
      break;
    case '\\":
      processEscape();
      break;
    default:
      processSomethingElse();
}

-Stepan.