[PATCH] D60106: AMDGPU: Assume ECC is enabled by default if supported

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 1 17:33:46 PDT 2019


arsenm created this revision.
arsenm added reviewers: rampitec, kzhuravl, scott.linder.
Herald added subscribers: eraman, t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, jvesely.

The test should really be checking for the property directly in the
code object headers, but there are problems with this. I don't see
 this directly represented in the text form, and for the binary
emission this is depending on a function level subtarget feature to
emit a global flag.


https://reviews.llvm.org/D60106

Files:
  lib/Target/AMDGPU/AMDGPU.td
  lib/Target/AMDGPU/AMDGPUSubtarget.cpp
  lib/Target/AMDGPU/AMDGPUSubtarget.h
  test/CodeGen/AMDGPU/load-hi16.ll
  test/CodeGen/AMDGPU/sram-ecc-default.ll
  test/Transforms/Inline/AMDGPU/inline-target-feature-sram-ecc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60106.193208.patch
Type: text/x-patch
Size: 9006 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190402/b3d9615d/attachment.bin>


More information about the llvm-commits mailing list