================ @@ -498,6 +498,7 @@ LLVM_ABI void dropDebugUsers(Instruction &I); /// /// The moved instructions receive the insertion point debug location values /// (DILocations) and their debug intrinsic instructions are removed. +/// Selects and indirect calls keep their MD_prof metadata. ---------------- nikic wrote:
I think it's actually fine to always handle MD_prof in dropUBImplyingAttrsAndMetadata() (that is push this one level down so you don't need to deal with this in each caller). My concern was only about doing it inside dropUnknownNonDebugMetadata(). https://github.com/llvm/llvm-project/pull/154635 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits