[clang] [clang][bytecode] Yet another __builtin_constant_p implementation (PR #130143)

Nikolas Klauser via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 7 00:00:55 PST 2025


philnik777 wrote:

I think this makes sense to me. However, it's entirely unclear to me how you prevent modifications for cases like
```c++
bool test(int& i) {
  return __builtin_constant_p(i = 5); // How does this fail?
}
```

> Okay, this fixes 112 libc++ tests.

BTW, this is awesome! The rate at which the libc++ test failures are going down makes me think we might be able to switch to the new interpreter in a year or so. I'm really interested how much of a test speedup we can get from this.

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


More information about the cfe-commits mailing list