Bug 236859 - [Tools] Sanitizer builds should obey `set-webkit-configuration --force-optimization-level`
Summary: [Tools] Sanitizer builds should obey `set-webkit-configuration --force-optimi...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
Keywords: InRadar
Depends on: 205787
  Show dependency treegraph
Reported: 2022-02-18 13:03 PST by David Kilzer (:ddkilzer)
Modified: 2022-02-18 23:20 PST (History)
9 users (show)

See Also:

Patch v1 (5.02 KB, patch)
2022-02-18 13:24 PST, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2022-02-18 13:03:44 PST
Sanitizer builds should obey `set-webkit-configuration --force-optimization-level` when set.

The reason it doesn't currently work that a command-line Xcode variable won't override the same variable from an xcconfig file specified on the command-line.
Comment 1 Radar WebKit Bug Importer 2022-02-18 13:04:17 PST
Comment 2 David Kilzer (:ddkilzer) 2022-02-18 13:24:24 PST
Created attachment 452571 [details]
Patch v1
Comment 3 David Kilzer (:ddkilzer) 2022-02-18 13:27:45 PST
Comment on attachment 452571 [details]
Patch v1

View in context: https://bugs.webkit.org/attachment.cgi?id=452571&action=review

> Tools/sanitizer/sanitizer.xcconfig:12
> +

I could copy this code into every `Base.xcconfig` file if we wanted to simplify the the `webkitdirs.pm` change to only set `WK_FORCE_OPTIMIZATION_LEVEL`, but I wasn't sure it was worth the extra complexity.

One advantage to making that change is that it makes it slightly clearer when the optimization level is being overridden, though you can still tell by noticing when `GCC_OPTIMIZATION_LEVEL` is set on the xcodebuild command-line.
Comment 4 Brent Fulgham 2022-02-18 16:40:52 PST
Comment on attachment 452571 [details]
Patch v1

I think this looks good as-is. We can consider moving things to Base.xcconfig if we need to do this elsewhere.
Comment 5 Aakash Jain 2022-02-18 16:56:40 PST
Cancelled https://ews-build.webkit.org/#/builders/70/builds/736 to speed up mac-wk2 queue. It already finished running layout-tests and the failures were pre-existing.
Comment 6 EWS 2022-02-18 23:20:50 PST
Committed r290199 (247527@main): <https://commits.webkit.org/247527@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 452571 [details].