[CMake] Remove forcing of -O2 from torch_compile_options#164894
[CMake] Remove forcing of -O2 from torch_compile_options#164894malfet wants to merge 2 commits intogh/malfet/547/basefrom
-O2 from torch_compile_options#164894Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/164894
Note: Links to docs will display an error until the docs builds have been completed. ⏳ No Failures, 73 PendingAs of commit 9f8a8b4 with merge base 600267e ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
malfet
added
ci-no-td
|
@pytorchbot merge -f "aarch64 builds are green this time around" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
…#164894) That was introduced by pytorch@75a65ff Hattip to @jathu for alerting me about the issue. As result, all our PyTorch builds were shipped with `-O2` for almost all of its modern history Partially undo the damage introduced by pytorch#128406 that cause cross-ISA symbols leak, to be properly followed up in pytorch#165123 Pull Request resolved: pytorch#164894 Approved by: https://github.com/ezyang
3 participants
Stack from ghstack (oldest at bottom):
-O2fromtorch_compile_options#164894That was introduced by 75a65ff
Hattip to @jathu for alerting me about the issue. As result, all our PyTorch builds were shipped with
-O2for almost all of its modern historyPartially undo the damage introduced by #128406 that cause cross-ISA symbols leak, to be properly followed up in #165123
cc @seemethere