[PATCH] D122425: Pass -disable-llvm-passes to avoid running llvm passes

xbolva00 added a comment. Thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122425/new/ https://reviews.llvm.org/D122425 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/

[PATCH] D122425: Pass -disable-llvm-passes to avoid running llvm passes

ahatanak added a comment. I removed `-O1` from the command line: https://github.com/llvm/llvm-project/commit/6ec79a15cbe9539faf121b5ad39f195dc611fc09 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122425/new/ https://reviews.llvm.org/D122425 __

[PATCH] D122425: Pass -disable-llvm-passes to avoid running llvm passes

ahatanak added a comment. Sorry, I didn't notice you left a comment until after committing the patch. I agree that `-O1` can be removed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122425/new/ https://reviews.llvm.org/D122425 __

[PATCH] D122425: Pass -disable-llvm-passes to avoid running llvm passes

xbolva00 added a comment. You should answer any review comments before any commit. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122425/new/ https://reviews.llvm.org/D122425 ___ cfe-commits mailing list

[PATCH] D122425: Pass -disable-llvm-passes to avoid running llvm passes

xbolva00 added inline comments. Comment at: clang/test/CodeGenCXX/nrvo.cpp:2 // NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py -// RUN: %clang_cc1 -triple i386-unknown-unknown -emit-llvm -O1 -o - %s | FileCheck %s -// RUN: %clang_cc1 -triple i386-unk

[PATCH] D122425: Pass -disable-llvm-passes to avoid running llvm passes

ahatanak added a comment. Yes, this test was susceptible to changes made to llvm passes and the pass pipeline. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122425/new/ https://reviews.llvm.org/D122425

[PATCH] D122425: Pass -disable-llvm-passes to avoid running llvm passes

Izaron accepted this revision. Izaron added a comment. This revision is now accepted and ready to land. 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