commit ac897edeb16a449c6e519e13ddf3decd8a6cb763
author Brian Osman <brianosman@google.com> Thu May 25 15:50:38 2023 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Fri May 26 14:51:58 2023 +0000
tree cd67dabcfeb27e497f7c13db9d9d391df3f4a6fb
parent 1622df0357140aa3a6114ab904e7a31bd0abd3a4

Revert "Omit frame pointers in SkOpts code."

This reverts commit 9be2676bc4e595bb7799c89370e99d9663da8c4b.

I don't think this actually did anything (for release builds). Frame
pointers are already omitted once optimization is enabled. (Note that we
do explicitly enable frame pointers for sanitizer builds, but the stack
quality improvements are worth it in that case).

Also: This was actually enabling frame pointers in Windows. /O2 implies
/Oy, which omits frame pointers. This was then adding /Oy-, which
disables omission of frame pointers.

Change-Id: I2122d2c065fbe194bfd48e5ae15b092f800322bd
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/703677
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>