Address CR comments

Author: Penguinwizzard

bin/NativeTests/CodexTests.cpp

@@ -3,10 +3,10 @@
 // Licensed under the MIT license. See LICENSE.txt file in the project root for full license information.
 //-------------------------------------------------------------------------------------------------------
 #include "stdafx.h"
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 #include <process.h>
 #include "Codex\Utf8Codex.h"

bin/NativeTests/FunctionExecutionTest.cpp

@@ -3,10 +3,10 @@
 // Licensed under the MIT license. See LICENSE.txt file in the project root for full license information.
 //-------------------------------------------------------------------------------------------------------
 #include "stdafx.h"
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 #include "FunctionExecutionTest.h"
 

bin/NativeTests/JsDiagApiTest.cpp

@@ -4,10 +4,10 @@
 //-------------------------------------------------------------------------------------------------------
 
 #include "stdafx.h"
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 #include <process.h>
 

bin/NativeTests/JsRTApiTest.cpp

@@ -3,10 +3,10 @@
 // Licensed under the MIT license. See LICENSE.txt file in the project root for full license information.
 //-------------------------------------------------------------------------------------------------------
 #include "stdafx.h"
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 #include <array>
 #include <process.h>

bin/NativeTests/MemoryPolicyTest.cpp

@@ -3,10 +3,10 @@
 // Licensed under the MIT license. See LICENSE.txt file in the project root for full license information.
 //-------------------------------------------------------------------------------------------------------
 #include "stdafx.h"
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 
 #pragma warning(disable:6387) // suppressing preFAST which raises warning for passing null to the JsRT APIs

bin/NativeTests/NativeTests.cpp

@@ -9,10 +9,10 @@
 // conversion from 'int' to 'char', possible loss of data
 #pragma warning(disable:4242)
 #pragma warning(disable:4244)
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 #pragma warning(pop)
 

bin/NativeTests/ThreadServiceTest.cpp

@@ -4,10 +4,10 @@
 //-------------------------------------------------------------------------------------------------------
 
 #include "stdafx.h"
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 
 #pragma warning(disable:6387) // suppressing preFAST which raises warning for passing null to the JsRT APIs

bin/NativeTests/UnicodeTextTests.cpp

@@ -4,10 +4,10 @@
 //-------------------------------------------------------------------------------------------------------
 
 #include "stdafx.h"
-#pragma warning(disable:26434)
-#pragma warning(disable:26439)
-#pragma warning(disable:26451)
-#pragma warning(disable:26495)
+#pragma warning(disable:26434) // Function definition hides non-virtual function in base class
+#pragma warning(disable:26439) // Implicit noexcept
+#pragma warning(disable:26451) // Arithmetic overflow
+#pragma warning(disable:26495) // Uninitialized member variable
 #include "catch.hpp"
 
 namespace UnicodeTextTest

lib/Backend/Lower.cpp

@@ -21156,7 +21156,7 @@ Lowerer::GenerateArgOutForStackArgs(IR::Instr* callInstr, IR::Instr* stackArgsIn
 
 
 #if defined(_M_IX86)
-    // We get a compilation error on x86 due to assiging a negative to a uint
+    // We get a compilation error on x86 due to assigning a negative to a uint
     // TODO: don't even define this function on x86 - we Assert(false) anyway there.
     // Alternatively, don't define when INT_ARG_REG_COUNT - 4 < 0
     AssertOrFailFast(false);