dim added a comment.

In D69825#2071687 <https://reviews.llvm.org/D69825#2071687>, @dim wrote:

> Trying to reduce this now.


Unfortunately this turned out to be a red herring, as the test case got reduced 
by `creduce` to just:

  a() { b(""); }

The valgrind warnings are also different when you build clang with clang, 
instead of gcc as I did first. In that case, you get dozens of these instead:

  ==525651== Conditional jump or move depends on uninitialised value(s)
  ==525651==    at 0xBB1D71: SimplifyAndInst(llvm::Value*, llvm::Value*, 
llvm::SimplifyQuery const&, unsigned int) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0xBBF6D0: ThreadBinOpOverPHI(llvm::Instruction::BinaryOps, 
llvm::Value*, llvm::Value*, llvm::SimplifyQuery const&, unsigned int) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0xBB1F87: SimplifyAndInst(llvm::Value*, llvm::Value*, 
llvm::SimplifyQuery const&, unsigned int) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x13898E8: 
llvm::InstCombiner::visitAnd(llvm::BinaryOperator&) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x1365A6D: llvm::InstCombiner::run() (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x1367AEC: combineInstructionsOverFunction(llvm::Function&, 
llvm::InstCombineWorklist&, llvm::AAResults*, llvm::AssumptionCache&, 
llvm::TargetLibraryInfo&, llvm::DominatorTree&, 
llvm::OptimizationRemarkEmitter&, llvm::BlockFrequencyInfo*, 
llvm::ProfileSummaryInfo*, bool, unsigned int, llvm::LoopInfo*) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x1369369: 
llvm::InstructionCombiningPass::runOnFunction(llvm::Function&) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x121015B: 
llvm::FPPassManager::runOnFunction(llvm::Function&) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x3BA5188: (anonymous 
namespace)::CGPassManager::runOnModule(llvm::Module&) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x1210B3F: llvm::legacy::PassManagerImpl::run(llvm::Module&) 
(in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x19A503A: 
clang::EmitBackendOutput(clang::DiagnosticsEngine&, clang::HeaderSearchOptions 
const&, clang::CodeGenOptions const&, clang::TargetOptions const&, 
clang::LangOptions const&, llvm::DataLayout const&, llvm::Module*, 
clang::BackendAction, std::unique_ptr<llvm::raw_pwrite_stream, 
std::default_delete<llvm::raw_pwrite_stream> >) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)
  ==525651==    by 0x2500BD4: 
clang::BackendConsumer::HandleTranslationUnit(clang::ASTContext&) (in 
/home/dim/obj/llvm-llvmorg-10.0.0-53-gf79cd71e145-linux5-x86_64-ninja-rel-1/bin/clang-10)

I'm unsure how to continue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69825



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to