[PATCH] D58159: AMDGPU: Remove debugger related subtarget features

Konstantin Zhuravlyov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 21 11:20:07 PST 2019


kzhuravl added a comment.

I am in favor of removing these.  Clang will also need to be updated around here:
https://github.com/llvm-mirror/clang/blob/master/lib/Driver/ToolChains/AMDGPU.cpp#L41

I tried to do the same thing about 6 months ago, but got a push back from @t-tye. @t-tye  is it ok to remove these now?


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

https://reviews.llvm.org/D58159





More information about the llvm-commits mailing list