Adding a TODO-XArch to `Lowering::ContainCheckHWIntrinsic` to track https://github.com/dotnet/coreclr/issues/16497

2 files changed, 7 insertions, 1 deletions

diff --git a/src/jit/hwintrinsiccodegenxarch.cpp b/src/jit/hwintrinsiccodegenxarch.cpp
index 9a0363f6a1..a4fc9f1875 100644
--- a/src/jit/hwintrinsiccodegenxarch.cpp
+++ b/src/jit/hwintrinsiccodegenxarch.cpp
@@ -244,7 +244,7 @@ void CodeGen::genHWIntrinsic_FullRangeImm8(GenTreeHWIntrinsic* node, instruction
     else
     {
         // We emit a fallback case for the scenario when op3 is not a constant. This should normally
-        // happen when the intrinsic is called indirectly, such as via Reflection. However, it can
+        // only happen when the intrinsic is called indirectly, such as via Reflection. However, it can
         // also occur if the consumer calls it directly and just doesn't pass a constant value.
 
         const unsigned jmpCount = 256;
diff --git a/src/jit/lowerxarch.cpp b/src/jit/lowerxarch.cpp
index 376c3194d3..8eb9e164cf 100644
--- a/src/jit/lowerxarch.cpp
+++ b/src/jit/lowerxarch.cpp
@@ -2406,6 +2406,9 @@ void Lowering::ContainCheckHWIntrinsic(GenTreeHWIntrinsic* node)
                 break;
 
             default:
+                // TODO-XArch-CQ: Assert that this is unreached after we have ensured the relevant node types are
+                // handled.
+                //                https://github.com/dotnet/coreclr/issues/16497
                 break;
         }
     }
@@ -2426,6 +2429,9 @@ void Lowering::ContainCheckHWIntrinsic(GenTreeHWIntrinsic* node)
             }
 
             default:
+                // TODO-XArch-CQ: Assert that this is unreached after we have ensured the relevant node types are
+                // handled.
+                //                https://github.com/dotnet/coreclr/issues/16497
                 break;
         }
     }