llvm-project icon indicating copy to clipboard operation
llvm-project copied to clipboard

Fix support for 64-bit atomics on 32-bit SparcV9

Open brad0 opened this issue 1 year ago • 4 comments

// TODO: we ought to be able to support 64-bit atomics on 32-bit sparcv9, // but it hasn't been implemented in the backend yet.

https://github.com/llvm/llvm-project/pull/81655

brad0 avatar Feb 14 '24 07:02 brad0

cc @koachan

brad0 avatar Feb 14 '24 07:02 brad0

@koachan

brad0 avatar Mar 23 '24 05:03 brad0

Noted, but I think it'd be better to fix the issues for 64-bit target first for now...

koachan avatar Mar 23 '24 16:03 koachan

Noted, but I think it'd be better to fix the issues for 64-bit target first for now...

I understand what you mean in general. This is SparcV9, this is relevant for Solaris/sparcv9 which uses 64-bit kernel and such, 32-bit userland. I mean as opposed to the 32-bit target V7 / V8.

brad0 avatar Mar 24 '24 07:03 brad0

@koachan With the V8+ support I wonder if this can be fixed.

brad0 avatar Mar 24 '25 01:03 brad0

Probably need to teach LLVM about the existence of 64-bit registers first? IIRC the support so far was only limited to recognizing/emitting the correct flags (and parsing inline asm), but there's still no codegen changes.

koachan avatar Mar 25 '25 14:03 koachan

@koachan Ah, good point. I was not aware of that.

brad0 avatar Mar 26 '25 01:03 brad0