[clang] [llvm] [SystemZ] Add support for __builtin_setjmp and __builtin_longjmp (PR #116642)
Ulrich Weigand via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 6 16:01:54 PST 2024
uweigand wrote:
Hi @anoopkg6 , I had to revert this again as it was causing a number of issues:
- An unused variable warning fixed here: https://github.com/llvm/llvm-project/commit/3c47e63723b1aa9e76f30fc8d1acef9caf4ea783
- The fuchsia build bot failure - I'm not completely sure what causes this, but it suspect the problem is that you should be using `%clang_cc1` instead of `%clang` (which would be a better idea anyway)
- The expensive checks build bot failure - this indicated a real bug the generated MI, which you can reproduce by adding `-verify-machineinstrs` to the test case (it would be a good idea to add this to those tests anyway!)
You'll have to open a new PR with a patch including those fixes.
https://github.com/llvm/llvm-project/pull/116642
More information about the cfe-commits
mailing list