[PATCH] D101141: [SimplifyCFG] Preserve metadata when unconditionalizing branches (constant condition).

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 23 23:58:41 PDT 2021


Meinersbur added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/Local.cpp:151
       // Replace the conditional branch with an unconditional one.
       Builder.CreateBr(Dest1);
       Value *Cond = BI->getCondition();
----------------
Meinersbur wrote:
> jeroen.dobbelaere wrote:
> > Shouldn't we also take care of the metadata here ?
> Yes, this patch is about the fixing PR50060 and I don't have a test case for this one.
See D101226


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101141/new/

https://reviews.llvm.org/D101141



More information about the llvm-commits mailing list