[PATCH] D135462: [SelectionDAG] Do not second-guess alignment for alloca

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 31 14:26:21 PDT 2023


efriedma added a comment.

> I've bisected a broken test in libcxx for the mingw/x86_64 target down to this commit - see https://github.com/llvm/llvm-project/issues/64253 for a full bug report including a reduced reproducer.

Most likely exposing an issue elsewhere.  All this patch does is change the alignment of allocas, so most likely there's some unrelated problem related to or exposed by the stack layout.  (Or maybe some other optimization is making bad assumptions about alloca alignment, but that's unlikely given the way the APIs in question work.)  Probably using opt-bisect-limit to bisect the exact optimization in question would help narrow down the issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135462



More information about the llvm-commits mailing list