Basic: use IndexType instead of unsigned

[compnerd]

Explicitly specify the aliased type (IndexType) rather than the
underlying type (unsigned) when invoking CreateNode to disambiguate
overload when targeting Windows. NFC.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

[compnerd]

@swift-ci please test and merge

[hughbe]

Sorry! I caused a merge conflict here - trying to resolve it through the GitHub UI to ease your work didn't work :(

[hughbe]

@swift-ci please smoke test

[swift-ci]

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 44ffe01
Test requested by - @hughbe

[swift-ci]

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 44ffe01
Test requested by - @hughbe

[hughbe]

@swift-ci please smoke test

[hughbe]

Here's a simple reproduction:

void addChild(uint64_t) { std::cout << "1"; }
void addChild(const char*) { std::cout << "2"; }

int main() { addChild(unsigned(0)); }

This looks like a Clang bug - GCC/MSVC reject this correctly.

[hughbe]

Opened http://bugs.llvm.org/show_bug.cgi?id=32073, obviously a breaking change - I suggested a warning instead of an error

[compnerd]

@swift-ci please test and merge

[compnerd]

@swift-ci please test OS X platform

[compnerd]

@swift-ci please test Linux platform

[swift-ci]

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - edc3031
Test requested by - @compnerd

[compnerd]

@swift-ci please test Linux platform