chandlerc wrote: @dyung - OK, I think the current just-pushed version of this PR is worth another test.
I've taught the TableGen string table emission to go back to working around the MSVC issues using a different table form that we used to use in LLVM when MSVC had a reliable error on it. It appears that now it only sometimes gets miscompiled, and so this workaround got accidentally removed without really being in a reliable place. But that still requires that all of the large string tables go through TableGen to get emitted in this form. So I've sent two PRs (separately, they seem independently good) to move NVPTX and Hexagon to TableGen, and this PR contains patches to move more of the ARM builtins to TableGen. I *think* this gets all of the ones that produced too-large string tables, but testing should show. Let me know. If this works and there isn't a really short path to upgrade, I'll start pulling some of the pre-requisite PRs out for independent review, and then update this PR once I'm down to the minimal chain of PRs that do the full conversion of builtins to string tables. Thanks, -Chandler https://github.com/llvm/llvm-project/pull/120534 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
