[clang] [Clang] Add "extend lifetime" flags and release note (PR #110000)
Jeremy Morse via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 29 07:53:31 PST 2024
jmorse wrote:
The additions here look fine; however I think there's generally precedent that anything we add needs to have /some/ kind of test, or be covered by something already existing, otherwise we're vulnerable to:
* Patch lands,
* Someone refactors clang switch handling,
* Other patches land but the flag doesn't do anything / is rejected,
* Confusion reigns
As far as I understand it, these are driver options that will be passed through to cc1? In that case we can at least test the passthrough, i.e. that `-fextend-lifetimes` appears in the output of `-###`.
https://github.com/llvm/llvm-project/pull/110000
More information about the cfe-commits
mailing list