[2/N] Use std::in_place

[cyyever]

Remove c10/util/in_place.h

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/115480

• 📄 Preview Python docs built from this PR
• 📄 Preview C++ docs built from this PR
• ❓ Need help or want to give feedback on the CI? Visit the bot commands wiki or our office hours

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (1 Unrelated Failure)

As of commit 71332ca with merge base 937d616 ():

FLAKY - The following job failed but was likely due to flakiness present on trunk:

• pull / linux-focal-py3.8-clang10-onnx / test (default, 2, 2, linux.2xlarge) (gh)
  onnx/test_fx_to_onnx_with_onnxruntime.py::TestFxToOnnxWithOnnxRuntime_op_level_debug_True_dynamic_shapes_False_model_type_TorchModelType.TORCH_EXPORT_EXPORTEDPROGRAM::test_sympy_operatons_return_numeric

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[cyyever]

@pytorchbot label "topic: not user facing"

[cyyever]

@pytorchbot label "ciflow/binaries"

[albanD]

Was this header file ever used since it's never included?

[soulitzer]

I think usages where it is included were removed in a previous PR by @cyyever
There are internal usages however that need to be addressed https://www.internalfb.com/diff/D52123195

[albanD]

Ok!
I'll let you land both @soulitzer !

[soulitzer]

@pytorchbot merge -i

[pytorch-bot]

This PR needs to be approved by an authorized maintainer before merge.

[soulitzer]

@pytorchbot merge -i

[pytorchmergebot]

Merge started

Your change will be merged while ignoring the following 1 checks: pull / linux-focal-py3.8-clang10-onnx / test (default, 2, 2, linux.2xlarge)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging

Check the merge workflow status
here

[pytorchmergebot]

Merge failed

Reason: 1 mandatory check(s) failed. The first few are:

• .github/workflows/pull.yml / linux-focal-py3.8-clang10-onnx / test (default, 2, 2, linux.2xlarge)

Dig deeper by viewing the failures on hud

Details for Dev Infra team

Raised by workflow job

Failing merge rule: Core Maintainers

[soulitzer]

@pytorchbot merge -f

[pytorch-bot]

❌ 🤖 pytorchbot command failed:

@pytorchbot merge: error: argument -f/--force: expected one argument

usage: @pytorchbot merge [-f MESSAGE | -i] [-ic] [-r [{viable/strict,main}]]

Try @pytorchbot --help for more info.

[soulitzer]

@pytorchbot merge -f "Unrelated failure"

[pytorchmergebot]

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging

Check the merge workflow status
here

[osalpekar]

@soulitzer This is breaking a large number of contbuilds due to internal usage of c10::in_place. Can you check them here and forward-fix: D52178758? Either that or we'll need to revert

[cyyever]

Use regex to replace 'c10::in_place' with 'std::in_place' should fix them.

[soulitzer]

@osalpekar Updated the diff!