[PATCH] D92842: [RFC][SelectionDAG] Add Target-Independent Compiler Barrier

Thu Dec 10 11:07:02 PST 2020

jrtc27 added inline comments.

================
Comment at: llvm/test/CodeGen/X86/atomic-idempotent.ll:384
+; X86-ATOM-NEXT:    # Compiler Barrier: acquire
+; X86-ATOM-NEXT:    nop
+; X86-ATOM-NEXT:    nop
----------------
craig.topper wrote:
> jrtc27 wrote:
> > What's up with this change?
> I'm guessing that TargetOpcode::COMPILER_BARRIER is giving a different value from TargetSchedMode::computeInstrLatency() that X86::MEMBARRIER did which changed how X86PadShortFunctions.cpp computed how many NOPs to add before the return.
I guess the question then is "which is better?".

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92842