Adds overload of ransformToPotentiallyEvaluated for TypeSourceInfo to properly deal with VLAs in nested calls of sizeof and typeof. Fixes #31042.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Apologies for the delay over the holiday season, I will look into this later on after office hours.
Comment Actions
Here's an updated patch including the test. Hope this is now ok for submission. Please accept my apologies with regards to the delay in submitting this.
Comment Actions
@efriedma I have uploaded a new patch taking your comments into consideration and rebased on most recent clang sources.
test/SemaCXX/pr31042.cpp | ||
---|---|---|
2 | Oh, this testcase doesn't actually crash on trunk without at least -emit-llvm because semantic analysis doesn't actually verify the used bit. :( Better to include that, I think. |
test/SemaCXX/pr31042.cpp | ||
---|---|---|
1 | You need to use "-o -" or something like that to avoid generating a file pr31042.ll. Also, a comment explaining why this test is using -emit-llvm would be nice. |