[llvm] [ExtendLifetimes] Implement llvm.fake.use to extend variable lifetimes (PR #86149)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 03:00:50 PDT 2024


https://github.com/OCHyams commented:

> Since python is already a dependency for running the tests I don't see a problem with it. While I am not sure we have tests implemented in python for clang, lldb has a lot of them so it's not unprecedented. I am interested in hearing what the objections would be in this case.

I can't speak for @dwblaikie but my objection was based on the fact that I would personally rather see a more brittle test that is simpler. It's not an objection against using python, just a question of whether it's really needed here.

That said, I don't feel strongly enough to insist and can see the benefit in having a slightly fuzzy test. Especially if it's going to be fiddly due to different debug info emission coming out of SelectionDAG/GlobalISel.

Does the script handle exprloc `DW_AT_location`s (i.e., single location that implicitly covers the scope with no explicit range)?


https://github.com/llvm/llvm-project/pull/86149


More information about the llvm-commits mailing list