diff options
| author | Omar Ahmed <omar.ahmed@codeplay.com> | 2025-05-28 14:40:00 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-05-28 07:40:00 -0600 |
| commit | 06ee672fc5222009d38fa8334b0b5438645f2a66 (patch) | |
| tree | 14b3b234b72626ee481c00d745c759fa545bf855 /clang/lib/AST/ByteCode/Interp.cpp | |
| parent | 59b7b5b6b5c032ed21049d631eb5d67091f3a21c (diff) | |
| download | llvm-06ee672fc5222009d38fa8334b0b5438645f2a66.zip llvm-06ee672fc5222009d38fa8334b0b5438645f2a66.tar.gz llvm-06ee672fc5222009d38fa8334b0b5438645f2a66.tar.bz2 | |
[clang] Move opt level in clang toolchain to clang::ConstructJob start (#141036)
We currently transfer the opt level from the user clang call to CC1 args
at the end of the `ConstructJob` function, this might lead to bugs as
`ConstructJob` is a big function and we easily could add a change that
would return early from it. That would cause the opt level to not be
transferred to CC1 args and lead to wrong opt level compilation and
would be hard to spot. This PR moves the opt level to the beginning of
the function as opt level should be a direct transfer without any
problems, it also removes the redundancy where it was added 2 times
through the function.
Diffstat (limited to 'clang/lib/AST/ByteCode/Interp.cpp')
0 files changed, 0 insertions, 0 deletions