[PATCH] D141777: [X86] Reenable store merging post-legalization
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 19 05:14:08 PST 2023
lebedev.ri marked an inline comment as done.
lebedev.ri added inline comments.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:674-675
+ return TargetLoweringBase::StoreSource::Load;
default:
- return StoreSource::Unknown;
+ return TargetLoweringBase::StoreSource::Unknown;
}
----------------
pengfei wrote:
> I'd agree with @barannikov88. `Unknown` and optional represent the same meaning. Why don't return `std::nullopt` here too?
The literally next change will return `splat` here instead.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141777/new/
https://reviews.llvm.org/D141777
More information about the llvm-commits
mailing list