It doesn't seem that clang/test/CodeGenCXX/nrvo.cpp needs llvm optimization passes.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Hi! I don't actually know much about llvm passes, but I see that the LLVM IR output is much more understandable now, thanks! It will look nicer when I rebase my patch D119792 on top of this (I just checked the diff without llvm passes)
clang/test/CodeGenCXX/nrvo.cpp | ||
---|---|---|
2 | Do you need -O1 at all? |
Comment Actions
Sorry, I didn't notice you left a comment until after committing the patch. I agree that -O1 can be removed.
Comment Actions
I removed -O1 from the command line:
https://github.com/llvm/llvm-project/commit/6ec79a15cbe9539faf121b5ad39f195dc611fc09
Do you need -O1 at all?