search for: thefutur

...proach seems to address most of the previous problems. I'm pretty uncomfortable with the proposed intrinsic. It feels tacked-on and not in the LLVM spirit. We should be able to extend the IR to support multiple targets. We're going to need this kind of support for much more than GPUs in thefuture. Heterogenous computing is here to stay. -Dave

...most of the previous problems. > > I'm pretty uncomfortable with the proposed intrinsic. It feels > tacked-on and not in the LLVM spirit. We should be able to extend the > IR to support multiple targets. We're going to need this kind of > support for much more than GPUs in thefuture. Heterogenous computing is > here to stay. Where exactly do you see problems with this intrinsic? It is not meant to block further work in heterogeneous computing, but to allow us to gradually improve LLVM to gain such features. It especially provides a low overhead solution that adds work...

...most of the previous problems. > > I'm pretty uncomfortable with the proposed intrinsic. It feels > tacked-on and not in the LLVM spirit. We should be able to extend the > IR to support multiple targets. We're going to need this kind of > support for much more than GPUs in thefuture. Heterogenous computing is > here to stay. > For me, the bigger question is: do we extend the IR to support multiple targets, or do we keep the one-target-per-module philosophy and derive some other way of representing how the modules fit together? I can see pros and cons for both approac...

On 04/28/2012 10:25 AM, Yabin Hu wrote: > Hi Justin, > > Thanks very much for your comments. > > 2012/4/28 Justin Holewinski <justin.holewinski at gmail.com > <mailto:justin.holewinski at gmail.com>> > > On Fri, Apr 27, 2012 at 7:40 PM, Yabin Hu <yabin.hwu at gmail.com > <mailto:yabin.hwu at gmail.com>> wrote: > > The

Hi Justin, Thanks very much for your comments. 2012/4/28 Justin Holewinski <justin.holewinski at gmail.com> > On Fri, Apr 27, 2012 at 7:40 PM, Yabin Hu <yabin.hwu at gmail.com> wrote: > >> The attached patch adds a new Intrinsic named "llvm.codegen" to support >> embedded LLVM IR code generation. **The 'llvm.codegen' intrinsic uses >> the