[PATCH] X86: rework expansion of atomic instructions

Tim Northover t.p.northover at gmail.com
Mon Jun 30 10:54:38 PDT 2014


Hi Eric,

> +bool X86AtomicExpandPass::needsCmpXchgNb(llvm::Type *MemType) {
> +  const X86Subtarget &Subtarget = TM->getSubtarget<X86Subtarget>();
> +  if (!Subtarget.hasCmpxchg16b())
> ----------------
> Is there any way to get this information from TTI instead? Relying on the subtarget here is going to go badly soon.

Should be fairly trivial, as long as a TTI can query the subtarget.
I'm fairly sure they can at the moment, but is that planned to go away
too? I'll get on it anyway.

Cheers.

Tim.



More information about the llvm-commits mailing list