[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