[PATCH] D149212: [MCParser] Add warning for zero-size .fill

[Jacob Young via Phabricator via llvm-commits]

jacobly marked an inline comment as done.
jacobly added inline comments.


================
Comment at: llvm/lib/MC/MCParser/AsmParser.cpp:3357-3364
+  if (FillSize == 0) {
+    Warning(SizeLoc, "'.fill' directive with 0 size has no effect");
+    return false;
+  }
   if (FillSize < 0) {
     Warning(SizeLoc, "'.fill' directive with negative size has no effect");
     return false;
----------------
arsenm wrote:
> Could you just have one message with the value instead 
Good idea.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149212