[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1e4e2433bcd1: [WebAssembly] Update supported features in the generic CPU configuration (authored by sunfish). Changed prior to commit: https://rev

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

sbc100 added a comment. Emscripten-side change in preparation for this to land: https://github.com/emscripten-core/emscripten/pull/17689 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125728/new/ https://reviews.llvm.org/D125728 __

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

sbc100 accepted this revision. sbc100 added a comment. This revision is now accepted and ready to land. Do you think we should backport this to llvm 15, since it missed the branch point? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125728/new/ ht

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

asb added a comment. One thing that's not ideal about this test case is that it still passes if you delete the changes to WebAssembly.cpp. At least for RISC-V, selecting the -mcpu also emits the relevant `-target-feature +foo` for features supported by that CPU, which doesn't seem to happen her

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

sunfish updated this revision to Diff 431067. sunfish added a comment. Add tests for -mcpu=mvp and -mcpu=bleeding-edge. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125728/new/ https://reviews.llvm.org/D125728 Files: clang/docs/ReleaseNotes.rst

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

sunfish updated this revision to Diff 431064. sunfish added a comment. Add a driver test, and add release notes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125728/new/ https://reviews.llvm.org/D125728 Files: clang/docs/ReleaseNotes.rst clan

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

asb added a comment. Oh, and per recent updates to the LLVM Developer policy I think it would be worth updating the Clang ReleaseNotes.rst to mention this change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION http

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

asb added a comment. Based on the discussion we had, I think this makes sense. It's a bit repetitive, but could you please add a test to clang/test/Driver that checks the list of enabled features for generic (and for completeness, probably bleeding-edge as well). Thanks. Repository: rG LLVM

[PATCH] D125728: [WebAssembly] Update supported features in -mcpu=generic

sunfish created this revision. Herald added subscribers: pmatos, asb, ecnelises, jgravelle-google, sbc100, dschuff. Herald added a project: All. sunfish requested review of this revision. Herald added a subscriber: aheejin. Herald added a project: clang. Enable nontrapping-fptoint, sign-ext, bulk