[PATCH] D69932: [IR] Redefine Freeze instruction

Mon Nov 11 14:20:02 PST 2019

craig.topper accepted this revision.
craig.topper added a comment.

In D69932#1740114 <https://reviews.llvm.org/D69932#1740114>, @lebedev.ri wrote:

> Thanks! Sorry for double work..
>
> I guess in principle this has backwards compatibility issues,
>  since `bitc::UNOP_FREEZE` goes away? But since LLVM itself
>  never produced them until this moment, do we care?
>
> Other than that this looks good to me.
>  Anyone else?

I don't think we have a backwards compatibility guarantee on trunk. Only across release boundaries. So I think this should be fine.

LGTM

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69932