================ @@ -719,60 +730,83 @@ decodeBBAddrMapImpl(const ELFFile<ELFT> &EF, Feature = Data.getU8(Cur); // Feature byte if (!Cur) break; - auto FeatEnableOrErr = PGOAnalysisMap::Features::decode(Feature); + auto FeatEnableOrErr = BBAddrMap::Features::decode(Feature); if (!FeatEnableOrErr) return FeatEnableOrErr.takeError(); - FeatEnable = - FeatEnableOrErr ? *FeatEnableOrErr : PGOAnalysisMap::Features{}; + FeatEnable = FeatEnableOrErr ? *FeatEnableOrErr : BBAddrMap::Features{}; ---------------- red1bluelost wrote:
(Nit) I think this can just always assume true given that we return on error in the line before. (woops, I think I wrote this wrong) ```suggestion FeatEnable = *FeatEnableOrErr; ``` https://github.com/llvm/llvm-project/pull/74128 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits