[PATCH] D29121: [Docs] Add LangRef documention for freeze instruction

[Nuno Lopes via Phabricator via llvm-commits]

nlopes added a comment.

In D29121#1725727 <https://reviews.llvm.org/D29121#1725727>, @reames wrote:

> In D29121#1724593 <https://reviews.llvm.org/D29121#1724593>, @nlopes wrote:
>
> > In D29121#1723978 <https://reviews.llvm.org/D29121#1723978>, @jdoerfert wrote:
> >
> > > So this is not committed till we reviewed all of them?
> >
> >
> > I'm just waiting on SelDAG->MIR patch to be done (https://reviews.llvm.org/D29014).
> >  Then we have docs, SelDAG & MIR support. That seems the minimal to me (so the compiler won't crash when freeze is used).  I agree the optimizer patches can land later.
>
>
> Can I suggest that we add a stub* ISEL implementation, land this, and then build on top of it?
>
> - By which I mean a knowingly incorrect lowering to a simple COPY.  It wouldn't fix *all* of our poison/undef problems, but it would allow us to make progress on some of the most painful IR ones while waiting for that patch to land.


Makes sense. We are trying to get commit access to Juneyoung, but we were caught in the middle of the GitHub transition :) It's unclear what the procedure is now -- working on it.


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

https://reviews.llvm.org/D29121