================
@@ -265,6 +279,14 @@ bool WebAssemblyTargetInfo::handleTargetFeatures(
       HasReferenceTypes = false;
       continue;
     }
+    if (Feature == "+call-indirect-overlong") {
+      HasCallIndirectOverlong = true;
+      continue;
+    }
+    if (Feature == "-call-indirect-overlong") {
+      HasCallIndirectOverlong = false;
+      continue;
+    }
----------------
aheejin wrote:

Nit: 
I understand why this was put after `reference-types`, because this is a subset 
of that, 
but the feature names here and elsewhere are sorted by the feature name's 
alphabetical order so I think `-call-indirect-overlong` could go right after 
`-bulk-memory-opt`. All other places too. Yeah it's probably my OCD 😅 But 
having them sorted helped me find missing tings easier...

https://github.com/llvm/llvm-project/pull/117087
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to