Update abilist for MacOS C++20

Ping

Update formatting to match the rest of the file

• Remove handling of INFINITY and associated tests, add a few tests for variations of INF

Ping

• Update ABI symbols to account for added parameter in stage2_float_loop
• Explicit cast in tests to avoid error when compiling with GCC

typo

That makes sense, you are correct that that warning is not prevented. I'm not sure what I did differently when I checked. LGTM.

It looks like the clang-tidy CI is broken (running clang tidy on backends that have been modified but are not built?). The rest of the CI looks ok so I will merge this.

clang-format

Rebase

Also ping

Replace lambda in header with macro

Update with new names: shouldRealignStack/canRealignStack/hasStackRealignment

needsStackRealignment feels too strong given that the target can say we're not realigning the stack anyway. How about something like hasImpliedStackRealignment? And a partner function to call both called something like hasStackRealignment? And a partner function to call both called something like hasStackRealignment?

Ping

Thank you everyone for the comments, they have been very useful. From the discussions here and internally it seems that neither of these approaches (new target hook for spill location, or using bundles) is the right way forward, and I should look into the pseudo instructions. Since this review has grown quite large I will abandon it and open a new review when I have something working with pseudo expansion.

Tidy up tests

• Add tests for double and long double
• Replace lambda with macro (there are C++03 builds, and filename/line number are not output when using lambda)
• Remove constexpr (C++03 builds)
• Undo refactoring of old tests

Minor change to tests

@lenary sorry for the delay in responding to comments.

• Address review comments and clang-tidy
• Rename isLoadInstr/isStoreInstr to isExclusiveLoad/isExclusiveStore

SparcFrameLowering: fix the check for failure to realign that was causing the test failure, and remove the old comment explaining needStackRealignment behavior.

@zoecarver yes sorry about that, for some reason I had it in my head that they were green. I'll keep an eye out in case I broke something.

Thanks. Should I wait for @ldionne to approve?

Disable fast path for clang < 13

Remove char and wchar_t tests

Ping

This seems quite an early point to bail out, so it will prevent some errors/warnings associated with initialization from being emitted. For example, this warning is currently emitted but would be suppressed by this patch:

This is great, thanks

...use SIZE_TYPE

int -> unsigned long long

Attempt to fix error on windows: change unsigned long -> int in operator new

• Add blank line after retun

was this causing problems for you in the wild, or did you just notice that there was an issue? If the latter, I think we should just take the tests from this patch and only enable them on ToT clang for the next six months.

• Add #ifdef around the __is_unsigned condition based on clang version number. I'm not convinced the value of this outweighs the noise it creates. But with it, this patch can be landed before or after D98104.

make libc++ conditionally use just __is_unsigned or a combination of __is_unsigned && !__is_enum based on the version of Clang

Ping

Ping. @arsenm, any comments on new approach using bundles, and the regalloc changes?

Thanks both, I'm still new to this process so didn't realise there were specific approvers.

@Quuxplusone @Mordante are there any outstanding issues to be addressed, or any further comments?

In D97283#2582217, @Mordante wrote:

Why doesn't is_signed be updated in the same way? I see the builds on buildkite passing, but the difference between is_unsigned and is_signed look very odd to me.

Update the comments on #else and #endif

Add tests for enums with explicitly signed and unsigned underlying types

Add corresponding tests to is_signed.pass.cpp

Ping

This is quite a substantial change in approach, please take a look.

This committed as b9ed8ebe0e2ffa803b0bda60f9bbc9bb26f95000

Use Optional rather than returning 0 (==TargetOpcode::PHI).
Rename isExclusiveLoad to IsExclusiveLoad.

Ping

Duplicate

Duplicate

Make unions in test cases const, and clarify logic

Add test RUN line that checks C++11 behaviour

Added more test cases and explanatory comments

• Added check for incompatible SHF_TLS flags
• Moved test to llvm/test/CodeGen/Generic
• Removed arm vendor from triple in test
• Changed isDefaultELFSection to check section by name rather than pointer value