tsc does not use amd-module name when emitting dynamic imports

<!-- 🚨 STOP 🚨 𝗦𝗧𝗢𝗣 🚨 𝑺𝑻𝑶𝑷 🚨 Half of all issues filed here are duplicates, answered in the FAQ, or not appropriate for the bug tracker. Even if you think you've found a *bug*, please read the FAQ first, especially the Common "Bugs" That Aren't Bugs section! Please help us by doing the following steps before logging an issue: * Search: https://github.com/Microsoft/TypeScript/search?type=Issues * Read the FAQ: https://github.com/Microsoft/TypeScript/wiki/FAQ Please fill in the *entire* template below. --> <!-- Please try to reproduce the issue with `typescript@next`. It may have already been fixed. --> **TypeScript Version:** 3.3.0-dev.20190118 <!-- Search terms you tried before logging this (so others can find this issue more easily) --> **Search Terms:** amd-module, optional, dynamic, directive **Code** Scripts/foo.ts ```ts ///<amd-module name="customNameFoo" /> export function hello() { console.log('hello world'); } ``` Scripts/bar.ts ```ts ///<amd-module name="customNameBar" /> async function myFunction(): Promise<void> { //Dynamically import foo var fooModule = await import('./foo'); fooModule.hello(); } ``` tsconfig.json ```json { "compileOnSave": true, "compilerOptions": { "target": "es5", "outDir": "GeneratedScripts", "module": "amd", "lib": [ "es2015.promise", "dom", "es5", "scripthost" ] }, "include": [ "Scripts/**/*" ], "exclude": [ "GeneratedScripts" ] } ``` would expect that the call to require would use the amd-module name for foo ("customNameFoo") and not "./foo". (If we non-dynamically imported foo, then the generated 'define' would refer to foo by it's amd-module name) GeneratedScripts/bar.js ```js //... return __generator(this, function (_a) { switch (_a.label) { case 0: return [4 /*yield*/, new Promise(function (resolve_1, reject_1) { require(['./foo'], resolve_1, reject_1); })]; case 1: fooModule = _a.sent(…