[PATCH] D71259: AMDGPU: Stop setting attributes based on TargetOptions

Tue Dec 10 09:23:10 PST 2019

rampitec added a comment.

In D71259#1777599 <https://reviews.llvm.org/D71259#1777599>, @arsenm wrote:

> In D71259#1777574 <https://reviews.llvm.org/D71259#1777574>, @rampitec wrote:
>
> > > clang and lld do not call the function to do this, which they maybe should.
> >
> > Should we first fix clang then?
>
>
> They probably shouldn't be using these flags, but I don't know where to diagnose it either. They can properly set the function attributes themselves. The purpose of these flags is to help writing lit tests

We should somehow translate opencl options into user code. I agree these should not be set on library functions. If clang properly set attributes now then this code indeed needs to be removed.

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

https://reviews.llvm.org/D71259