[PATCH] D123991: [LangRef] Clarify load/store of non-byte-sized types

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 24 18:58:00 PDT 2022


arsenm added a comment.

In D123991#3470465 <https://reviews.llvm.org/D123991#3470465>, @craig.topper wrote:

> In D123991#3467043 <https://reviews.llvm.org/D123991#3467043>, @lebedev.ri wrote:
>
>> As for `i1`, you have seen enum BooleanContent <https://github.com/llvm/llvm-project/blob/15650b320bf6a1ce5b7e00147d1cf2725946aab2/llvm/include/llvm/CodeGen/TargetLowering.h#L229-L234>, correct?
>
> BooleanContent is not involved in load/store handling for i1 types. See these comments in LegalizeDAG.cpp

BooleanContent is only used in boolean contexts, not that we have a definition for what that means. It's used only for compare results and select/branch conditions


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

https://reviews.llvm.org/D123991



More information about the llvm-commits mailing list