[llvm] [BOLT] Turn non-empty CFI StateStack assert into a warning (PR #102216)
Davide Italiano via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 6 14:13:09 PDT 2024
https://github.com/dcci commented:
Thanks for working on this. I don't understand CFI enough to comment on the correctness of this, hopefully @rafaelauler can chime in.
As a general point, I'm not entirely sure we should disable this assertion, without understanding.
In particular:
1) Is this something that the compiler produced that's corrupted?
Should that be the case, we might consider either:
a) fixing the compiler
b) making this an error
c) allow this to be a warning (if someone passes a flag to override at their own risk)
If 1) doesn't hold, is BOLT corrupting the stack somehow?
https://github.com/llvm/llvm-project/pull/102216
More information about the llvm-commits
mailing list