mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-10-20 11:33:24 +02:00
4f3ea7d288
It used to be that all of our intrinsics were call instructions, but over time, we've added more and more invokable intrinsics. According to the verifier, we're up to 8 right now. As IntrinsicInst is a sub-class of CallInst, this puts us in an awkward spot where the idiomatic means to check for intrinsic has a false negative if the intrinsic is invoked.
This change switches IntrinsicInst from being a sub-class of CallInst to being a subclass of CallBase. This allows invoked intrinsics to be instances of IntrinsicInst, at the cost of requiring a few more casts to CallInst in places where the intrinsic really is known to be a call, not an invoke.
After this lands and has baked for a couple days, planned cleanups:
Make GCStatepointInst a IntrinsicInst subclass.
Merge intrinsic handling in InstCombine and use idiomatic visitIntrinsicInst entry point for InstVisitor.
Do the same in SelectionDAG.
Do the same in FastISEL.
Differential Revision: https://reviews.llvm.org/D99976
|
||
|---|---|---|
| .. | ||
| IntelJITEvents | ||
| Interpreter | ||
| JITLink | ||
| MCJIT | ||
| OProfileJIT | ||
| Orc | ||
| PerfJITEvents | ||
| RuntimeDyld | ||
| CMakeLists.txt | ||
| ExecutionEngine.cpp | ||
| ExecutionEngineBindings.cpp | ||
| GDBRegistrationListener.cpp | ||
| SectionMemoryManager.cpp | ||
| TargetSelect.cpp | ||