================ @@ -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