show error for target lower than esnext with useddefineforclassfields enabled

Author: a-tarasyuk

src/compiler/checker.ts

@@ -2210,7 +2210,7 @@ m2: ${(this.mapper2 as unknown as DebugTypeMapper).__debugToString().split("\n")
             // 1. When result is undefined, after checking for a missing "this."
             // 2. When result is defined
             function checkAndReportErrorForInvalidInitializer() {
-                if (propertyWithInvalidInitializer && !useDefineForClassFields) {
+                if (propertyWithInvalidInitializer && !(useDefineForClassFields && getEmitScriptTarget(compilerOptions) >= ScriptTarget.ESNext)) {
                     // We have a match, but the reference occurred within a property initializer and the identifier also binds
                     // to a local variable in the constructor where the code will be emitted. Note that this is actually allowed
                     // with ESNext+useDefineForClassFields because the scope semantics are different.

tests/baselines/reference/classMemberInitializerScoping2(target=es2017,usedefineforclassfields=true).symbols

@@ -7,7 +7,6 @@ class C {
 
     p = x
 >p : Symbol(C.p, Decl(classMemberInitializerScoping2.ts, 1, 9))
->x : Symbol(x, Decl(classMemberInitializerScoping2.ts, 0, 5))
 
     constructor(x: string) { }
 >x : Symbol(x, Decl(classMemberInitializerScoping2.ts, 3, 16))

tests/baselines/reference/classMemberInitializerScoping2(target=es2017,usedefineforclassfields=true).types

@@ -7,8 +7,8 @@ class C {
 >C : C
 
     p = x
->p : number
->x : 1
+>p : any
+>x : any
 
     constructor(x: string) { }
 >x : string