[PATCH] D57737: [AMDGPU] Fix DPP sequence in atomic optimizer.

[David Stuttard via Phabricator via llvm-commits]

dstuttard added inline comments.


================
Comment at: llvm/trunk/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp:256
-  MDNode *const RegName =
-      llvm::MDNode::get(Context, llvm::MDString::get(Context, "exec"));
-  Value *const Metadata = llvm::MetadataAsValue::get(Context, RegName);
----------------
dnovillo wrote:
> This is leaving the declaration for |Context| unused in line 214.  I'm getting build errors with -Wunused-variable.
There's a later change that removes it. See  https://llvm.org/svn/llvm-project/llvm/trunk@353704 by Benny Kramer.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57737