[PATCH] D68544: [X86][AVX] Access a scalar float/double as a free extract from a broadcast load (PR43217)

Denis Antrushin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 14 11:20:59 PDT 2019


dantrushin added a comment.

Sure, but it will take some time to minimize it etc.
Meantime, my original comment was wrong - basically, I have these DAG nodes:

  t5: i16,ch = AtomicLoad<(load unordered 2 from %ir.51, align 8)> t0, t4, Test::iMeth-482306:0
  ...
   t589: v8i16 = X86ISD::VBROADCAST t5, Test::iMeth-482306:0
  ...
   t679: v8i32,ch = X86ISD::VBROADCAST_LOAD<(load 4 from %ir.52)> t5:1, t30, Test::iMeth-482306:0

t5:1 here is a chain, so we apparently combine unrelated nodes here?
Looks like VBROADCAST_LOAD needs a bit more checks?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68544/new/

https://reviews.llvm.org/D68544





More information about the llvm-commits mailing list