[PATCH] D35974: AMDGPU: Add pass to replace out arguments

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 11:18:54 PDT 2017


rampitec added a comment.

In https://reviews.llvm.org/D35974#824310, @arsenm wrote:

> In https://reviews.llvm.org/D35974#823655, @rampitec wrote:
>
> > LGTM, but I would suggest checking opt level before adding to the PM.
>
>
> I think I'll just drop adding the pass for now until I figure out where best to put it. Also it's probably better to put calls without this through more testing before enabling this by default


You can always add an option off by default.


https://reviews.llvm.org/D35974





More information about the llvm-commits mailing list