[llvm] r346366 - [AArch64] [Windows] Address post-commit review comment on r346358.

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 7 14:30:56 PST 2018


Author: efriedma
Date: Wed Nov  7 14:30:56 2018
New Revision: 346366

URL: http://llvm.org/viewvc/llvm-project?rev=346366&view=rev
Log:
[AArch64] [Windows] Address post-commit review comment on r346358.

In this context, usesWindowsCFI() is basically the same thing as
isOSWindows(), but it makes the relevant property of the target
more explicit.


Modified:
    llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp

Modified: llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp?rev=346366&r1=346365&r2=346366&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp (original)
+++ llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp Wed Nov  7 14:30:56 2018
@@ -30,6 +30,7 @@
 #include "llvm/CodeGen/TargetPassConfig.h"
 #include "llvm/IR/Attributes.h"
 #include "llvm/IR/Function.h"
+#include "llvm/MC/MCAsmInfo.h"
 #include "llvm/MC/MCTargetOptions.h"
 #include "llvm/Pass.h"
 #include "llvm/Support/CodeGen.h"
@@ -263,7 +264,7 @@ AArch64TargetMachine::AArch64TargetMachi
     this->Options.NoTrapAfterNoreturn = true;
   }
 
-  if (TT.isOSWindows()) {
+  if (getMCAsmInfo()->usesWindowsCFI()) {
     // Unwinding can get confused if the last instruction in an
     // exception-handling region (function, funclet, try block, etc.)
     // is a call.




More information about the llvm-commits mailing list