Details
Diff Detail
Event Timeline
lib/CodeGen/GlobalISel/IRTranslator.cpp | ||
---|---|---|
875 | Can we use DL.getABITypeAlignment() here? |
lib/CodeGen/GlobalISel/IRTranslator.cpp | ||
---|---|---|
875 | Probably? I don't know anything about vastart. I'm not sure why this has an MMO in the first place |
lib/CodeGen/GlobalISel/IRTranslator.cpp | ||
---|---|---|
875 | vastart's argument is a target dependent pointer to a va_list object on the stack, so I think it's worth it to try to get the right alignment. That said, 1 is always safe so I think it's ok to use for now. |
lib/CodeGen/GlobalISel/IRTranslator.cpp | ||
---|---|---|
875 | I've looked at what SelectionDAG does for this, and I'm still not clear on what to use. The targets are creating the MMO there, and each one seems to be doing something different. Most of the cases seem to create a MachinePointerInfo and then get*Store functions magically infer an alignment from the underlying IR value. Some targets are hardcoding this. I think this is a separate patch |
lib/CodeGen/GlobalISel/IRTranslator.cpp | ||
---|---|---|
875 | Ok, fine. |
Can we use DL.getABITypeAlignment() here?