[PATCH] D76973: [LangRef] Clarify the semantics of branch on undef

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 29 12:17:57 PDT 2020


nlopes added a comment.

In D76973#1948751 <https://reviews.llvm.org/D76973#1948751>, @spatel wrote:

> In D76973#1948172 <https://reviews.llvm.org/D76973#1948172>, @fhahn wrote:
>
> > We should probably also think about what to do with all the tests that contain branches on undef. Also, bugpoint should probably stop creating branches on undef with this spelled out as is?
>
>
> When I've updated other tests that would break with changed/improved undef semantics, I usually replace the 'undef' with a parameter like this:
>  rGfebcb24f1490 <https://reviews.llvm.org/rGfebcb24f14901ed4b666533a02d099dccd511201>
>  Maybe we can script that to fix most of the tests?


agreed!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76973





More information about the llvm-commits mailing list