[PATCH] D94867: [ARM] Make a BE predicate bitcast consistent with the rest of llvm

Mon Jan 18 03:03:12 PST 2021

markus added a comment.

> Yes, but that is base ARM and it is only MVE that is incorrect. You can see here that things were inconsistent, which is what this patch is fixing:
> https://godbolt.org/z/M8Y6dv

Yes. I agree. As long was we know which is the right behavior.

> It comes from https://reviews.llvm.org/D42100#992315. This patch just brings MVE inline with what the rest of llvm expects.

Thank you! I have been trying to find where this decision was taken. But now that we know that shouldn't we make an effort to document this in the lang-ref document (or is it already and simply couldn't find it)? It seems like something that is quite fundamental to be aware of and not at all obvious.

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

https://reviews.llvm.org/D94867