Drop obsolete code from std to llvm memref lowering

tensorflow / mlir

"Multi-Level Intermediate Representation" Compiler Infrastructure

1.74k stars 259 forks source link

Drop obsolete code from std to llvm memref lowering #179

Closed bondhugula closed 5 years ago

bondhugula commented 5 years ago

dropping what looks like outdated code post some of the previous updates

Signed-off-by: Uday Bondhugula uday@polymagelabs.com

joker-eph commented 5 years ago

@bondhugula can you fix this? Thanks!

lib/Conversion/StandardToLLVM/ConvertStandardToLLVM.cpp:608:10: error: unused variable 'numOperands' [-Werror,-Wunused-variable]
    auto numOperands = allocOp.getNumOperands();
         ^
1 error generated.

bondhugula commented 5 years ago

@bondhugula can you fix this? Thanks!

lib/Conversion/StandardToLLVM/ConvertStandardToLLVM.cpp:608:10: error: unused variable 'numOperands' [-Werror,-Wunused-variable]
    auto numOperands = allocOp.getNumOperands();
         ^
1 error generated.

Yes, I actually fixed that while the build check was running. It should be fine now.