[PATCH] D32330: X86: Don't emit zero-byte functions on Windows

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 10:23:01 PDT 2017


hans added inline comments.


================
Comment at: lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1056
-    // unspecified.
-    if (Noop.getOpcode())
-      OutStreamer->EmitInstruction(Noop, getSubtargetInfo());
----------------
I checked, and there are no in-tree targets that "opt out" in this way.
It would be awkward, because getNoopForMachoTarget() is also used for other things, e.g. ARMAsmPrinter::EmitSled.


https://reviews.llvm.org/D32330





More information about the llvm-commits mailing list