[llvm] [X86] Support ATOMIC_LOAD_FP_BINOP_MI for other binops (PR #87524)

Phoebe Wang via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 22 23:03:04 PDT 2024


https://github.com/phoebewang commented:

> @phoebewang What do you think about this?

I don't know the background, just looked two patches https://github.com/llvm/llvm-project/commit/8662083770b34a7e62c8a3ae01f6b2e5be417417 and https://github.com/llvm/llvm-project/commit/586fad50ac47fbb110737ce7ea2f503c609472ab.

My guess it's a bit complex for the first patch to support them. The second one simplified it and should expand to others. I don't see problem here, but maybe @topperc should take a look.

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


More information about the llvm-commits mailing list