Fix LLVMRustSetHasUnsafeAlgebra to only have effect on instructions

bluss · bluss · commit e22d6d569fcb · 2016-03-19T22:35:28.000+01:00
diff --git a/src/rustllvm/RustWrapper.cpp b/src/rustllvm/RustWrapper.cpp
@@ -165,8 +165,10 @@ extern "C" void LLVMRemoveFunctionAttrString(LLVMValueRef fn, unsigned index, co
 }
 
 // enable fpmath flag UnsafeAlgebra
-extern "C" void LLVMRustSetHasUnsafeAlgebra(LLVMValueRef Instr) {
-    unwrap<Instruction>(Instr)->setHasUnsafeAlgebra(true);
+extern "C" void LLVMRustSetHasUnsafeAlgebra(LLVMValueRef V) {
+    if (auto I = dyn_cast<Instruction>(unwrap<Value>(V))) {
+        I->setHasUnsafeAlgebra(true);
+    }
 }
 
 extern "C" LLVMValueRef LLVMBuildAtomicLoad(LLVMBuilderRef B,

Original file line number	Diff line number	Diff line change
`@@ -165,8 +165,10 @@ extern "C" void LLVMRemoveFunctionAttrString(LLVMValueRef fn, unsigned index, co`
`165`	`165`	`}`
`166`	`166`
`167`	`167`	`// enable fpmath flag UnsafeAlgebra`
`168`		`-extern "C" void LLVMRustSetHasUnsafeAlgebra(LLVMValueRef Instr) {`
`169`		`- unwrap<Instruction>(Instr)->setHasUnsafeAlgebra(true);`
	`168`	`+extern "C" void LLVMRustSetHasUnsafeAlgebra(LLVMValueRef V) {`
	`169`	`+ if (auto I = dyn_cast<Instruction>(unwrap<Value>(V))) {`
	`170`	`+ I->setHasUnsafeAlgebra(true);`
	`171`	`+ }`
`170`	`172`	`}`
`171`	`173`
`172`	`174`	`extern "C" LLVMValueRef LLVMBuildAtomicLoad(LLVMBuilderRef B,`