[llvm-commits] [llvm] r146692 - in /llvm/trunk/lib/Target/Hexagon: CMakeLists.txt Hexagon.h HexagonInstrInfo.cpp HexagonMCAsmInfo.cpp HexagonMCAsmInfo.h HexagonRegisterInfo.cpp HexagonSubtarget.cpp HexagonTargetMachine.cpp LLVMBuild.txt MCTargetD

[Sebastian Pop]

Hi Jakob,

On Thu, Dec 15, 2011 at 5:39 PM, Jakob Stoklund Olesen <stoklund at 2pi.dk> wrote:
>
> On Dec 15, 2011, at 2:29 PM, Tony Linthicum wrote:
>
>> Author: tlinth
>> Date: Thu Dec 15 16:29:08 2011
>> New Revision: 146692
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=146692&view=rev
>> Log:
>> Add MCTargetDesc library to Hexagon target
>
> Hi Tony,
>
> I am still seeing segfaults when trying to use the Hexagon target on OS X:
>
> $ Release+Asserts/bin/llc -march=hexagon -mcpu=hexagonv4 < /d/g/llvm/test/CodeGen/Hexagon/combine.ll
> 0  llc               0x0000000105702e52 _ZL15PrintStackTracePv + 34
> 1  llc               0x0000000105703369 _ZL13SignalHandleri + 697
> 2  libsystem_c.dylib 0x00007fff91d2dcfa _sigtramp + 26
> 3  libsystem_c.dylib 0x0000000105d57258 _sigtramp + 18446603342462489976
> 4  llc               0x0000000104e4e745 llvm::HexagonTargetMachine::HexagonTargetMachine(llvm::Target const&, llvm::StringRef, llvm::StringRef, llvm::StringRef, llvm::TargetOptions, llvm::Reloc::Model, llvm::CodeModel::Model, llvm::CodeGenOpt::Level) + 325
> 5  llc               0x0000000104e4eb94 llvm::RegisterTargetMachine<llvm::HexagonTargetMachine>::Allocator(llvm::Target const&, llvm::StringRef, llvm::StringRef, llvm::StringRef, llvm::TargetOptions const&, llvm::Reloc::Model, llvm::CodeModel::Model, llvm::CodeGenOpt::Level) + 196
> 6  llc               0x0000000104de7724 main + 3044
> 7  llc               0x0000000104de6b34 start + 52
> Stack dump:
> 0.      Program arguments: Release+Asserts/bin/llc -march=hexagon -mcpu=hexagonv4
> Segmentation fault: 11
>
> If you don't have a Mac available, I would suggest that you try to figure out what the Hexagon target is doing different compared to the other targets when initializing the TargetMachine instance.
>

Could you please try to remove the call to setMCUseCFI from the Hexagon
initialization function and report if that fixes your problem?

Thanks,
Sebastian

> /jakob
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits


--
Qualcomm Innovation Center, Inc is a member of Code Aurora Forum
-------------- next part --------------
diff --git a/lib/Target/Hexagon/HexagonTargetMachine.cpp b/lib/Target/Hexagon/HexagonTargetMachine.cpp
index b29e92c..d95a2eb 100644
--- a/lib/Target/Hexagon/HexagonTargetMachine.cpp
+++ b/lib/Target/Hexagon/HexagonTargetMachine.cpp
@@ -60,7 +60,6 @@ HexagonTargetMachine::HexagonTargetMachine(const Target &T, StringRef TT,
     TSInfo(*this),
     FrameLowering(Subtarget),
     InstrItins(&Subtarget.getInstrItineraryData()) {
-  setMCUseCFI(false);
 }
 
 // addPassesForOptimizations - Allow the backend (target) to add Target