[PATCH] D152362: [llvm-lib] Pass the right value of MingwDef to parseCOFFModuleDefinition

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 7 04:40:39 PDT 2023


mstorsjo created this revision.
mstorsjo added reviewers: hans, vadikp-intel, alvinhochun.
Herald added a subscriber: hiraditya.
Herald added a project: All.
mstorsjo requested review of this revision.
Herald added a project: LLVM.

In llvm-lib cases, the MinGW specific behaviours shouldn't be
enabled.

Not adding any tests to check the corner case behaviours, as this
simply was a mistake when this codepath was added in
395ec4458fb7fc700551f7017c0a395d68c55873 <https://reviews.llvm.org/rG395ec4458fb7fc700551f7017c0a395d68c55873> / D144765 <https://reviews.llvm.org/D144765>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D152362

Files:
  llvm/lib/ToolDrivers/llvm-lib/LibDriver.cpp


Index: llvm/lib/ToolDrivers/llvm-lib/LibDriver.cpp
===================================================================
--- llvm/lib/ToolDrivers/llvm-lib/LibDriver.cpp
+++ llvm/lib/ToolDrivers/llvm-lib/LibDriver.cpp
@@ -384,7 +384,7 @@
     }
 
     Expected<COFFModuleDefinition> Def =
-        parseCOFFModuleDefinition(*MB, LibMachine, true);
+        parseCOFFModuleDefinition(*MB, LibMachine, /*MingwDef=*/false);
 
     if (!Def) {
       llvm::errs() << "error parsing definition\n"


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D152362.529253.patch
Type: text/x-patch
Size: 489 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230607/5c0d07c9/attachment.bin>


More information about the llvm-commits mailing list