forked from OSchip/llvm-project
[WebAssembly][SelectionDAG] Remove unused WebAssemblyDAGToDAGISel::ForCodeSize.
Summary: This follows from the discussion at D70095. D70095 moves hasOptSize calls into SelectionDAG::shouldOptForSize to allow querying size optimization conditions together with profile guided size optimization. Since it appears that size optimizations for WebAssembly SelectionDAG haven't been implemented yet and thus ForCodeSize is unused, and it would not make a lot of sense to call shouldOptForSize here as the necessary profile data like PSI/BFI aren't available at this point, it seems good and less confusing to remove this for now and use shouldOptForSize when they are implemented in the future. Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, aheejin, sunfish, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D70567
This commit is contained in:
parent
5459a905c2
commit
70a3c9f55f
|
@ -35,12 +35,10 @@ class WebAssemblyDAGToDAGISel final : public SelectionDAGISel {
|
|||
/// right decision when generating code for different targets.
|
||||
const WebAssemblySubtarget *Subtarget;
|
||||
|
||||
bool ForCodeSize;
|
||||
|
||||
public:
|
||||
WebAssemblyDAGToDAGISel(WebAssemblyTargetMachine &TM,
|
||||
CodeGenOpt::Level OptLevel)
|
||||
: SelectionDAGISel(TM, OptLevel), Subtarget(nullptr), ForCodeSize(false) {
|
||||
: SelectionDAGISel(TM, OptLevel), Subtarget(nullptr) {
|
||||
}
|
||||
|
||||
StringRef getPassName() const override {
|
||||
|
@ -52,7 +50,6 @@ public:
|
|||
"********** Function: "
|
||||
<< MF.getName() << '\n');
|
||||
|
||||
ForCodeSize = MF.getFunction().hasOptSize();
|
||||
Subtarget = &MF.getSubtarget<WebAssemblySubtarget>();
|
||||
|
||||
// Wasm64 is not fully supported right now (and is not specified)
|
||||
|
|
Loading…
Reference in New Issue