[llvm] [DXIL] Add constraint specification and backend implementation of DXIL Ops (PR #97593)

Chris B via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 12 14:04:44 PDT 2024


================
@@ -124,32 +117,29 @@ DXILOperationDesc::DXILOperationDesc(const Record *R) {
   // the comment before the definition of class LLVMMatchType in
   // llvm/IR/Intrinsics.td
   SmallVector<int> OverloadParamIndices;
-  for (unsigned i = 0; i < TypeRecsSize; i++) {
-    auto TR = TypeRecs[i];
+  for (unsigned I = 0; I < ParamTypeRecsSize; I++) {
+    auto TR = ParamTypeRecs[I];
     // Track operation parameter indices of any overload types
-    auto isAny = TR->getValueAsInt("isAny");
-    if (isAny == 1) {
+    auto IsAny = TR->getValueAsInt("isAny");
+    if (IsAny == 1) {
----------------
llvm-beanz wrote:

There are so many NFC changes that clean up this code (which is great) but grouping those in an already large PR makes it a lot harder to review.

https://github.com/llvm/llvm-project/pull/97593


More information about the llvm-commits mailing list