[PATCH] D80715: [ARM] prologue instructions emitted for naked function with >64 byte argument

Brian J. Cardiff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 17 17:57:51 PDT 2020


bcardiff added a comment.

I notice this change caused a regression described in https://bugs.llvm.org/show_bug.cgi?id=47708 . I guess there is something in the LowerArguments that is actually needed for the naked functions. I am not familiar with the SelectionDAGISel implementation nor to how to iterate and try fixes that will not require a 1h+ compilation. If someone could guide I can try to fix this. But more importantly, I would like to get pass the regression so I can update llvm to 11 in the crystal language.

To describe how unfamiliar I am with this: I bisected between 11.0.0 and 11-init, which took 13 iterations with 1h+ and 20gb each step. I don't know better. I checked that reverting this change on top of 11.0.0 fixes the issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80715



More information about the llvm-commits mailing list